Skip to content

ci: verify renovate config#7792

Merged
achrinza merged 1 commit intomasterfrom
ci/renovate-config-check
Aug 27, 2021
Merged

ci: verify renovate config#7792
achrinza merged 1 commit intomasterfrom
ci/renovate-config-check

Conversation

@achrinza
Copy link
Member

@achrinza achrinza commented Aug 26, 2021

Renovate doesn't provide any feedback on a malformed config before merging. This new status check will act as a safeguard and prevent bad configs from being merged into the default branch.

Signed-off-by: Rifa Achrinza 25147899+achrinza@users.noreply.github.com

Checklist

  • DCO (Developer Certificate of Origin) signed in all commits
  • npm test passes on your machine
  • New tests added or existing tests modified to cover all changes
  • Code conforms with the style guide
  • API Documentation in code was updated
  • Documentation in /docs/site was updated
  • Affected artifact templates in packages/cli were updated
  • Affected example projects in examples/* were updated

👉 Check out how to submit a PR 👈

@achrinza achrinza requested a review from raymondfeng as a code owner August 26, 2021 12:03
@achrinza achrinza force-pushed the ci/renovate-config-check branch from 212481e to 3a5402d Compare August 26, 2021 12:17
Signed-off-by: Rifa Achrinza <25147899+achrinza@users.noreply.github.com>
@achrinza achrinza force-pushed the ci/renovate-config-check branch from 3a5402d to f77ab10 Compare August 27, 2021 08:40
@coveralls
Copy link

coveralls commented Aug 27, 2021

Pull Request Test Coverage Report for Build 1173587118

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 75.955%

Totals Coverage Status
Change from base Build 1172824978: 0.0%
Covered Lines: 16887
Relevant Lines: 17876

💛 - Coveralls

@achrinza achrinza merged commit 3ffd2a6 into master Aug 27, 2021
@achrinza achrinza deleted the ci/renovate-config-check branch August 27, 2021 13:41
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