Skip to content

docs(arrow): add complete list of accepted kwargs to shift() docstring#1307

Open
koteshyelamati wants to merge 1 commit into
arrow-py:masterfrom
koteshyelamati:patch-2
Open

docs(arrow): add complete list of accepted kwargs to shift() docstring#1307
koteshyelamati wants to merge 1 commit into
arrow-py:masterfrom
koteshyelamati:patch-2

Conversation

@koteshyelamati

Copy link
Copy Markdown

Addresses #1287: the shift() docstring now explicitly lists all accepted keyword arguments: years, months, weeks, days, hours, minutes, seconds, microseconds, quarters, weekday. Each param is documented with its type/description using Sphinx :param: format.

Pull Request Checklist

Thank you for taking the time to improve Arrow! Before submitting your pull request, please check all appropriate boxes:

  • 🧪 Added tests for changed code.
  • 🛠️ All tests pass when run locally (run tox or make test to find out!).
  • 🧹 All linting checks pass when run locally (run tox -e lint or make lint to find out!).
  • 📚 Updated documentation for changed code.
  • ⏩ Code is up-to-date with the master branch.

If you have any questions about your code changes or any of the points above, please submit your questions along with the pull request and we will try our best to help!

Description of Changes

Addresses arrow-py#1287: the shift() docstring now explicitly lists all accepted keyword arguments:
years, months, weeks, days, hours, minutes, seconds, microseconds, quarters, weekday.
Each param is documented with its type/description using Sphinx :param: format.
@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (2224255) to head (cd3fe47).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #1307   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines         2315      2315           
  Branches       358       358           
=========================================
  Hits          2315      2315           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants