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: lint #21

Closed
wants to merge 2 commits into from
Closed

chore: lint #21

wants to merge 2 commits into from

Conversation

corang
Copy link
Contributor

@corang corang commented Feb 5, 2024

lint project and update pre-commit

@corang corang requested a review from a team as a code owner February 5, 2024 16:07
mjnagel
mjnagel previously approved these changes Feb 5, 2024
Comment on lines +61 to +68
{
"fileMatch": [".*\\.ya?ml$"],
"matchStrings": [
"# renovate: repo-file depName=(?<depName>.*?)( versioning=(?<versioning>.*?))?\\n.*\\/(?<currentValue>v.*?)\\/"
],
"versioningTemplate": "{{#if versioning}}{{{versioning}}}{{else}}semver-coerced{{/if}}",
"datasourceTemplate": "github-tags"
},
Copy link
Contributor

Choose a reason for hiding this comment

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

I would recommend we don't do this. This is going to be very messy long term and instead we should pick a single renovate comment format.

Copy link
Contributor

Choose a reason for hiding this comment

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

As of #20 this repo now uses this renovate configuration: https://github.com/defenseunicorns/uds-common/blob/main/config/renovate.json5

This common rule will match this dependency without needing a # renovate comment at all: https://github.com/defenseunicorns/uds-common/blob/main/config/renovate.json5#L30

Copy link
Contributor

Choose a reason for hiding this comment

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

@Racer159
Copy link
Contributor

closing as completed in #41

@Racer159 Racer159 closed this Mar 21, 2024
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