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 issue with wrong until in GitHub search #376

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Felixoid
Copy link

@Felixoid Felixoid commented Oct 9, 2024

It fixes #362.

A decoupled part from #363

@psss
Copy link
Owner

psss commented Oct 9, 2024

Thanks for fixing this! Would you mind extending the /tests/github/issues and /tests/github/pulls tests with a simple test case demonstrating that the dates now work as expected?

@psss psss added this to the 0.22 milestone Oct 9, 2024
@psss psss self-assigned this Oct 9, 2024
@Felixoid
Copy link
Author

Felixoid commented Oct 9, 2024

The PRs and issues appearing when running tests against the master:

teemtee/tmt#1645, teemtee/tmt#1644, teemtee/tmt#1648, teemtee/tmt#1650

@Felixoid
Copy link
Author

It was previously inconsistent with using somewhere defined local variables and somewhere not. Fixed it

@Felixoid
Copy link
Author

This PR is ready to review and merge. Since it touches the same code, I'll use it as a base for #363.

@Felixoid
Copy link
Author

Felixoid commented Nov 8, 2024

Dear @psss, I'd like to have it merged before the #363. That PR uses the changed until and touches the code close to the same as here. Do you have time for a quick review?

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

Successfully merging this pull request may close these issues.

Too recent GitHub pull requests included
2 participants