Skip to content
New issue

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

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

Already on GitHub? # 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?

@Felixoid
Copy link
Author

Hey-hey, happy new year!

A kind ping?

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

Successfully merging this pull request may close these issues.

Too recent GitHub pull requests included
2 participants