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

Resolve a semantic conflict between #5608 and #5628. #5660

Merged
merged 1 commit into from
Apr 27, 2023

Conversation

jswrenn
Copy link
Contributor

@jswrenn jswrenn commented Apr 27, 2023

This PR resolves a semantic conflict between #5608 and #5628. Both PRs modified tokio/src/runtime/task/list.rs:

Because these changes did not occur close together, they did not pose a merge conflict for either PR; rather, a semantic conflict.

This PR fixes the semantic conflict, but does not fix that CI allows such conflicts to occur. To resolve that, this repo would need to adopt something like bors.

Further discussion: https://discord.com/channels/500028886025895936/500336346770964480/1100809662208675982

@Darksonn Darksonn added A-tokio Area: The main tokio crate M-runtime Module: tokio/runtime A-ci Area: The continuous integration setup labels Apr 27, 2023
Copy link
Contributor

@Darksonn Darksonn left a comment

Choose a reason for hiding this comment

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

Thanks.

@Darksonn Darksonn merged commit f478ff4 into tokio-rs:master Apr 27, 2023
@hawkw
Copy link
Member

hawkw commented Apr 27, 2023

GitHub branch protection's "Require pull requests to be up-to-date" setting would also have prevented this, I believe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ci Area: The continuous integration setup A-tokio Area: The main tokio crate M-runtime Module: tokio/runtime
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants