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

👷 Remove Dependabot-triggered workflows #598

Merged
merged 4 commits into from
Sep 18, 2022

Conversation

TeoZosa
Copy link
Owner

@TeoZosa TeoZosa commented Apr 14, 2022

What

  1. Recouples Release, Publish ADR Documentation via Log4brains, & Reset Instance (in the meta-project) workflows to commits/PRs
  2. Couples Publish Performance Testing Benchmarks workflow to commits/PRs
  3. Removes now-unused Dependabot-triggered workflows and associated dependencies.

Why

Dependabot was previously unable to access repo secrets. This was rectified by (a) the introduction of a permissions key which Dependabot now honors, and (b) the addition of Dependabot secrets as a GitHub-native feature. This will now make CI more robust since workflows no longer run need to be launched in isolation, which also reduces unnecessary duplicate workflow runs for developer (non-Dependabot)-triggered workflows, and are coupled to triggered commits/PRs for review.

References

@TeoZosa TeoZosa self-assigned this Apr 14, 2022
@TeoZosa TeoZosa added ci Changes to CI configuration files and scripts cookiecutter Changes outside of the template directory github_actions Pull requests that update Github_actions code labels Apr 14, 2022
@TeoZosa TeoZosa force-pushed the ci/migrate-to-dependabot-secrets branch 3 times, most recently from 0288ef6 to f8fe89d Compare April 14, 2022 21:40
@TeoZosa TeoZosa force-pushed the ci/migrate-to-dependabot-secrets branch 2 times, most recently from 2c5af74 to 208b5f1 Compare April 29, 2022 04:42
@TeoZosa TeoZosa force-pushed the ci/migrate-to-dependabot-secrets branch from 208b5f1 to efb74c9 Compare September 2, 2022 00:14
Previously used to resolve issues with Dependabot's inability to access
repo secrets. This was rectified by the addition of Dependabot secrets
as a GitHub-native feature.
- see https://docs.github.com/en/code-security/dependabot/working-with-dependabot/managing-encrypted-secrets-for-dependabot
Previously used to resolve issues with Dependabot's inability to access
repo secrets. This was rectified by the addition of Dependabot secrets
as a GitHub-native feature.
- see https://docs.github.com/en/code-security/dependabot/working-with-dependabot/managing-encrypted-secrets-for-dependabot
To fix long-running Windows benchmarks failing CI.
@TeoZosa TeoZosa force-pushed the ci/migrate-to-dependabot-secrets branch from efb74c9 to a3e5f86 Compare September 18, 2022 05:49
@TeoZosa TeoZosa merged commit a168e64 into master Sep 18, 2022
@TeoZosa TeoZosa deleted the ci/migrate-to-dependabot-secrets branch September 18, 2022 06:01
TeoZosa added a commit that referenced this pull request Dec 14, 2022
Benchmarks publishing broken since #598.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Changes to CI configuration files and scripts cookiecutter Changes outside of the template directory github_actions Pull requests that update Github_actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant