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

[notify-debouncer-full] mark as compatible with rstest 0.18.x #556

Merged

Conversation

michel-slm
Copy link
Contributor

We are packaging this in Fedora, and tests pass using rstest 0.18.x that is in our repositories

See https://bugzilla.redhat.com/show_bug.cgi?id=2257209

@JohnTitor
Copy link
Member

I'd just declare rstest = "0.18", I don't think the flexibility is useful on the dev deps.

We are packaging this in Fedora, and tests pass using rstest 0.18.x that
is in our repositories

See https://bugzilla.redhat.com/show_bug.cgi?id=2257209

Signed-off-by: Michel Lind <[email protected]>
@michel-slm michel-slm force-pushed the notify-debouncer-full-rstest-widen-range branch from 6ad64cb to 272c3b3 Compare January 19, 2024 21:29
@michel-slm
Copy link
Contributor Author

I'd just declare rstest = "0.18", I don't think the flexibility is useful on the dev deps.

good point! updated the commit

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

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

Thank you!

@JohnTitor JohnTitor merged commit 24d9b4d into notify-rs:main Jan 20, 2024
13 checks passed
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