Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix doc on Days::new to refer to days, not months. #872

Closed
wants to merge 1 commit into from
Closed

Fix doc on Days::new to refer to days, not months. #872

wants to merge 1 commit into from

Conversation

brotskydotcom
Copy link
Contributor

This fixes #871

Thanks for contributing to chrono!

If your feature is semver-compatible, please target the 0.4.x branch;
the main branch will be used for 0.5.0 development going forward.

Please consider adding a test to ensure your bug fix/feature will not break in the future.

@djc djc changed the base branch from main to 0.4.x November 14, 2022 09:18
@djc djc changed the base branch from 0.4.x to main November 14, 2022 09:19
@djc
Copy link
Member

djc commented Nov 14, 2022

Thanks! Would you mind rebasing this on top of 0.4.x to target the 0.4.x branch with this PR, please?

@brotskydotcom brotskydotcom closed this by deleting the head repository Nov 14, 2022
@brotskydotcom
Copy link
Contributor Author

brotskydotcom commented Nov 14, 2022 via email

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.

doc for Days::new() refers to months, not days
2 participants