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

Fixes return types for the before and after query methods to match the documentation #561

Merged
merged 1 commit into from
Feb 10, 2023

Conversation

jkepps
Copy link
Contributor

@jkepps jkepps commented Nov 25, 2022

Addresses #560


Thanks for contributing to rrule!

To submit a pull request, please verify that you have done the following:

  • Merged in or rebased on the latest master commit
  • Linked to an existing bug or issue describing the bug or feature you're
    addressing
  • Written one or more tests showing that your change works as advertised

@jkepps
Copy link
Contributor Author

jkepps commented Nov 25, 2022

@davidgoli This PR addresses the comment you left on #462, and is slightly more comprehensive than #524

Copy link
Collaborator

@davidgoli davidgoli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution!

@davidgoli davidgoli merged commit 346b313 into jkbrzt:master Feb 10, 2023
@ghost ghost mentioned this pull request Sep 13, 2024
7 tasks
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