Skip to content

Conversation

@BurntSushi
Copy link
Member

There are already a fair number and I'm planning to add more. And
lock.rs is already quite big.

There aren't any new tests or other changes here. This is just moving
tests and trimming down the function names to avoid redundancy in the
names.

There are already a fair number and I'm planning to add more. And
`lock.rs` is already quite big.

There aren't any new tests or other changes here. This is just moving
tests and trimming down the function names to avoid redundancy in the
names.
@BurntSushi BurntSushi merged commit cc6bfa1 into main Nov 27, 2024
64 checks passed
@BurntSushi BurntSushi deleted the ag/move-conflict-tests branch November 27, 2024 15:51
BurntSushi added a commit that referenced this pull request Dec 2, 2024
Embarrassingly, PR #9474 moved the conflicting extras/groups tests into
their own module, but never actually included the module in
`it/main.rs`.

This adds `lock_conflict` to `main.rs` and fixes the fallout.
BurntSushi added a commit that referenced this pull request Dec 2, 2024
Embarrassingly, PR #9474 moved the conflicting extras/groups tests into
their own module, but never actually included the module in
`it/main.rs`.

This adds `lock_conflict` to `main.rs` and fixes the fallout.
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