Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Minor: only trigger dependency check on changes to Cargo.toml#10099

Merged
Jefffrey merged 3 commits into
apache:mainfrom
alamb:alamb/cleanup_dep_check
Apr 17, 2024
Merged

Minor: only trigger dependency check on changes to Cargo.toml#10099
Jefffrey merged 3 commits into
apache:mainfrom
alamb:alamb/cleanup_dep_check

Conversation

@alamb

@alamb alamb commented Apr 16, 2024

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Follow on to #9865

Related to #9278

Rationale for this change

It would be nice not to waste runner time (and burn CO2) on unecessary jobs

What changes are included in this PR?

Only trigger dependency check on changes to Cargo.toml

Are these changes tested?

I tested them manually:

Are there any user-facing changes?

@alamb alamb force-pushed the alamb/cleanup_dep_check branch 2 times, most recently from 916c0c0 to f106ff1 Compare April 16, 2024 16:25
@alamb alamb marked this pull request as ready for review April 16, 2024 16:26

@kavirajk kavirajk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM :)

@Jefffrey Jefffrey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

πŸ‘

@Jefffrey Jefffrey merged commit f715d8c into apache:main Apr 17, 2024
@alamb alamb deleted the alamb/cleanup_dep_check branch April 19, 2024 11:32
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.

3 participants