Skip to content

ci: add file hygiene workflow and resolve all lint errors#221

Closed
yshyn-iohk wants to merge 3 commits into
mainfrom
ci/file-hygiene
Closed

ci: add file hygiene workflow and resolve all lint errors#221
yshyn-iohk wants to merge 3 commits into
mainfrom
ci/file-hygiene

Conversation

@yshyn-iohk
Copy link
Copy Markdown
Contributor

Summary

Implements file hygiene linting for apollo as part of the org-wide rollout (hyperledger-identus/hyperledger-identus#172).

1. CI workflow and lint configuration

  • Add file-hygiene.yml caller workflow for the reusable lint-files.yml
  • Prepend canonical .editorconfig baseline to existing ktlint rules
  • Add .gitattributes, .markdownlint.yml, .markdownlint-cli2.yaml, .yamllint.yml
  • Excludes secp256k1-kmp/native/secp256k1/ submodule from all linters

2. Lint fixes across the repo (10 files)

  • Markdownlint auto-fix: trailing spaces, blank lines around headings/lists/fences
  • CRLF → LF normalization (DCO.md, MAINTAINERS.md, SECURITY.md)

3. .git-blame-ignore-revs

  • Add mass formatting commit to blame ignore list

Test plan

  • File Hygiene (editorconfig) — passing
  • Markdown — passing (0 errors locally)
  • YAML — passing
  • ShellCheck — passing

Refs: hyperledger-identus/hyperledger-identus#172

🤖 Generated with Claude Code

@yshyn-iohk yshyn-iohk requested a review from a team as a code owner April 13, 2026 17:11
@sonarqubecloud
Copy link
Copy Markdown

yshyn-iohk and others added 3 commits May 20, 2026 02:03
Add reusable file-hygiene workflow caller and canonical lint configs:

- .github/workflows/file-hygiene.yml: caller for lint-files.yml
- .editorconfig: UTF-8 (no BOM), LF line endings, indent rules
  with indent_size=4 override for Kotlin (*.kt, *.kts)
- .gitattributes: LF normalization for text files, binary rules
- .markdownlint.yml + .markdownlint-cli2.yaml: markdown lint rules
  with secp256k1-kmp/native/secp256k1/ excluded
- .yamllint.yml: YAML lint rules with relaxations

Refs: hyperledger-identus/hyperledger-identus#172

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Markdownlint auto-fix resolved formatting issues:
- MD009: trailing spaces
- MD012: multiple consecutive blank lines
- MD022: missing blank lines around headings
- MD031: missing blank lines around fenced code blocks
- MD032: missing blank lines around lists
- MD047: missing final newline

CRLF to LF normalization: DCO.md, MAINTAINERS.md, SECURITY.md

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
@sonarqubecloud
Copy link
Copy Markdown

@yshyn-iohk
Copy link
Copy Markdown
Contributor Author

Commits pushed directly to main to preserve 3-commit structure and correct .git-blame-ignore-revs SHA.

Commits on main:

  • 5f69a221e ci: add file hygiene workflow and lint configuration
  • 657843c5a style: auto-fix markdown formatting across the repo
  • ccee22bcd chore: add formatting commit to .git-blame-ignore-revs

CI green on main: https://github.com/hyperledger-identus/apollo/actions/runs/26119103816

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