Skip to content

chore: add pre-commit with linters#72

Merged
dcalavrezo-qorix merged 1 commit intomainfrom
linters
Mar 6, 2026
Merged

chore: add pre-commit with linters#72
dcalavrezo-qorix merged 1 commit intomainfrom
linters

Conversation

@AlexanderLanin
Copy link
Member

  • add pre-commit
  • add linters
  • add formatters
  • one change in codeql.yml: parameter cannot be required AND have a default

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds repository-level developer tooling (pre-commit + YAML formatting/linting) and aligns GitHub workflow YAML formatting, including a small CodeQL input fix.

Changes:

  • Add pre-commit configuration and yamlfmt settings (plus VS Code recommendations/settings).
  • Normalize/format multiple GitHub Actions workflow YAML files.
  • Fix CodeQL reusable-workflow input to not be both required and have a default.

Reviewed changes

Copilot reviewed 9 out of 15 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
.pre-commit-config.yaml Introduces pre-commit hooks (YAML checks, actionlint, yamlfmt).
.yamlfmt Adds yamlfmt configuration.
.vscode/settings.json Adds editor settings related to formatting/validation for YAML.
.vscode/extensions.json Recommends extensions for GitHub Actions + YAML.
.github/workflows/local_on_pr.yml Adds a PR workflow intended to run pre-commit in CI.
.github/workflows/codeql.yml Adjusts build-script input required/default combination.
.github/workflows/required-approvals.yml YAML formatting changes; updates a run: step representation.
.github/workflows/on-pr.yml Minor YAML formatting (branches list).
.github/workflows/score-pr-checks.yml Line-ending/format normalization only.
.github/workflows/docs-verify.yml Line-ending/format normalization only.
.github/workflows/docs-cleanup.yml Minor YAML formatting (permissions indentation).
.github/workflows/license-check.yml Removes stray whitespace-only lines.
.github/workflows/format.yml Removes stray whitespace-only line.
.github/workflows/copyright.yml Minor whitespace/indent normalization.
.github/actions/deploy-versioned-pages/action.yml Removes stray blank line and adjusts comment indentation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dcalavrezo-qorix dcalavrezo-qorix merged commit 5140f75 into main Mar 6, 2026
3 checks passed
@dcalavrezo-qorix dcalavrezo-qorix deleted the linters branch March 6, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants