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

ci: configure additional CI jobs & bring linter settings in compliance with OSS template #279

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

jtieri
Copy link
Member

@jtieri jtieri commented Aug 21, 2024

This PR adds some additional CI jobs:

  • Codespell (which checks for grammar mistakes)
  • Dependabot (which monitors for out of date deps and security vulnerabilities)
  • Mergify (which can be used for automatically backporting commits across maintained versions)
  • CodeQL (which looks for code smells, security vulnerabilities, etc.)
  • Markdown Link Checker (which looks for broken markdown hyperlinks)
  • Title Format (which enforces conventional commits in PR titles)

It also tweaks the golangci-lint settings to be in compliance with the OSS template.

All repo settings should also be in compliance with the OSS template now, besides for configuring the few extra CI jobs to have passing statues before PRs can be merged.


TODO: Do we bring the build & test CI job and the goreleaser CI job into compliance with the OSS template or do we leave the currently configured settings?

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.

1 participant