Skip to content

dependabot: fix ignoring substrate deps#1588

Merged
ascjones merged 6 commits intomasterfrom
aj/dependabot
Jan 11, 2023
Merged

dependabot: fix ignoring substrate deps#1588
ascjones merged 6 commits intomasterfrom
aj/dependabot

Conversation

@ascjones
Copy link
Copy Markdown
Collaborator

@ascjones ascjones commented Jan 11, 2023

The ignore config from #1561 don't seem to be working to prevent dependabot PRs.

This PR moves the ignore section to the cargo updates section as per the example in the docs: https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#ignore.

Also only ignores major updates since the automatic substrate releases process seems to do frequent major version bumps.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 11, 2023

Codecov Report

Merging #1588 (247593b) into master (cc76f0f) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1588      +/-   ##
==========================================
- Coverage   71.59%   71.54%   -0.05%     
==========================================
  Files         205      205              
  Lines        6298     6298              
==========================================
- Hits         4509     4506       -3     
- Misses       1789     1792       +3     
Impacted Files Coverage Δ
crates/metadata/src/lib.rs 45.00% <0.00%> (-10.00%) ⬇️
crates/allocator/src/bump.rs 87.60% <0.00%> (-0.83%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ascjones ascjones changed the title dependabot: explicitly ignore substrate deps dependabot: fix ignoring substrate deps Jan 11, 2023
@ascjones ascjones merged commit 76b8c91 into master Jan 11, 2023
@ascjones ascjones deleted the aj/dependabot branch January 11, 2023 13:15
HCastano pushed a commit that referenced this pull request Jan 23, 2023
* dependabot: explicitly ignore substrate deps

* Indentation

* Revert to wildcards, move to cargo package

* Comments

* Fix update-types

* Formatting
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.

4 participants