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

chore: add pre-commit-config.yaml #123

Merged
merged 10 commits into from
Dec 3, 2024
Merged

chore: add pre-commit-config.yaml #123

merged 10 commits into from
Dec 3, 2024

Conversation

cloudsmith-iduffy
Copy link
Contributor

Adding a pre-commit-config.yaml to enforce formatting and go checks

Ran pre-commit run --all-files as part of opening this PR so all automated fix are in place.

Copy link
Member

@lskillen lskillen left a comment

Choose a reason for hiding this comment

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

Looks great; I love me some pre-commit linting. :)

Copy link
Contributor

@BartoszBlizniak BartoszBlizniak left a comment

Choose a reason for hiding this comment

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

Thank you Ian!

…entials

chore(no-ticket): validate provided credentials in the plan phase
feat(ENG-6470): add support for configuring saml authentication
@cloudsmith-iduffy cloudsmith-iduffy merged commit 2c910df into master Dec 3, 2024
4 checks passed
@cloudsmith-iduffy cloudsmith-iduffy deleted the iduffy/chores branch December 3, 2024 11:48
@cloudsmith-iduffy cloudsmith-iduffy restored the iduffy/chores branch December 3, 2024 11:48
@cloudsmith-iduffy cloudsmith-iduffy deleted the iduffy/chores branch December 3, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants