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

dependency_cross_matcher job: Move to PR runs and minor refactoring #743

Conversation

tgonzalezorlandoarm
Copy link
Member

@tgonzalezorlandoarm tgonzalezorlandoarm commented Nov 27, 2023

bitflags crate was recently updated to 2.4.1 in the Cargo.lock files. This has shown potential improvements on the job, as a new mismatch appeared in the nightly when we could easily have discovered it in a normal CI run.

bitflags crate was recently updated to 2.4.1 in the Cargo.lock
files.

 * Update the exceptions list in dependency_cross_matcher.py to
   reflect said change.

Signed-off-by: Tomás González <[email protected]>
@tgonzalezorlandoarm tgonzalezorlandoarm added testing Task related to testing dependencies Pull requests that update a dependency file labels Nov 27, 2023
@tgonzalezorlandoarm tgonzalezorlandoarm self-assigned this Nov 27, 2023
Script was showing only the mismatches of the project but was not
showing the allowed dependencies, which made it hard to spot the
actual issue.

Signed-off-by: Tomás González <[email protected]>
The dependency mismatcher job should run on each PR to spot issues
as soon as possible.

Signed-off-by: Tomás González <[email protected]>
@tgonzalezorlandoarm tgonzalezorlandoarm changed the title utils/dependency_cross_matcher.py: Update bitflags exception to 2.4.1 dependency_cross_matcher job: Move to PR runs and minor refactoring Nov 27, 2023
Copy link
Contributor

@gowthamsk-arm gowthamsk-arm left a comment

Choose a reason for hiding this comment

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

Thanks. LGTM

@tgonzalezorlandoarm tgonzalezorlandoarm merged commit a78df06 into parallaxsecond:main Nov 27, 2023
16 checks passed
@tgonzalezorlandoarm tgonzalezorlandoarm deleted the tg/dep-cross-matcher-update branch November 27, 2023 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file testing Task related to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants