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): commit and label lint #709

Closed
Tracked by #727
justinthelaw opened this issue Jul 1, 2024 · 0 comments · Fixed by #931
Closed
Tracked by #727

chore(lint): commit and label lint #709

justinthelaw opened this issue Jul 1, 2024 · 0 comments · Fixed by #931
Assignees
Labels
tech-debt Not a feature, but still necessary

Comments

@justinthelaw
Copy link
Contributor

Describe what should be investigated or refactored

Linting for PR reviews is necessary to ensure consistent commit message creation, and standardized label application. Label application can be in the form of descriptive or corrective action relative to the PR's contents (e.g., enhancement for features, needs-docs for blocking merging when docs are not updated). Commit message creation is based on the PR title (based on repository rules), and therefore must be linted every time a PR is modified or created, else it will block merging.

Links to any relevant code

Example label lint: https://github.com/justinthelaw/uds-rke2/blob/main/.github/workflows/scan-labels.yaml
Example commit lint: https://github.com/justinthelaw/uds-rke2/blob/main/.github/workflows/commit-lint.yaml

Additional context

N/A

@justinthelaw justinthelaw added the tech-debt Not a feature, but still necessary label Jul 1, 2024
@justinthelaw justinthelaw changed the title chore(lint): commit and label lint chore(lint): commit and label docs lint Jul 1, 2024
@justinthelaw justinthelaw changed the title chore(lint): commit and label docs lint chore(lint): commit and label lint Jul 1, 2024
@justinthelaw justinthelaw linked a pull request Aug 7, 2024 that will close this issue
@justinthelaw justinthelaw self-assigned this Aug 8, 2024
@justinthelaw justinthelaw linked a pull request Aug 19, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech-debt Not a feature, but still necessary
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant