Skip to content

fix(ci): allow generated Turtle EOF blank lines - #372

Merged
pedropaulofb merged 1 commit into
masterfrom
fix/ci-generated-turtle-eof-whitespace
Sep 8, 2026
Merged

fix(ci): allow generated Turtle EOF blank lines#372
pedropaulofb merged 1 commit into
masterfrom
fix/ci-generated-turtle-eof-whitespace

Conversation

@pedropaulofb

Copy link
Copy Markdown
Member

Summary

The final PR-state validation was failing because Git's default blank-at-eof check rejected the established blank line emitted by the repository's generated Turtle files.

Changes

  • Allow generated Turtle files to contain a blank line at EOF during final PR validation.
  • Preserve checks for trailing spaces and spaces before tabs.
  • Add regression coverage confirming that generated EOF blank lines are accepted while real trailing whitespace is rejected.

No generated model artifacts or workflow files were modified.

Validation

  • 526 Python tests passed.
  • The exact PR Add leblanc2026rocap #370 base-to-head whitespace check passed.
  • Generated model outputs remained unchanged.
  • Catalog and release validation passed.
  • All workflow YAML files parsed successfully.

Preserve trailing-whitespace checks while allowing the repository's established RDF/Turtle serializer output. Add regression coverage for generated EOF blank lines and real trailing spaces.
@pedropaulofb
pedropaulofb deployed to pr-automation September 8, 2026 21:13 — with GitHub Actions Active
@pedropaulofb
pedropaulofb deployed to pr-automation September 8, 2026 21:14 — with GitHub Actions Active
@pedropaulofb
pedropaulofb merged commit 43cbcc7 into master Sep 8, 2026
6 checks passed
@pedropaulofb
pedropaulofb deleted the fix/ci-generated-turtle-eof-whitespace branch September 8, 2026 21:26
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