Skip to content

docs+ci: Boost documentation and test coverage for #358 - #359

Closed
laurentketterle-hub wants to merge 2 commits into
sourcey:mainfrom
laurentketterle-hub:boost-docs-ci-358
Closed

docs+ci: Boost documentation and test coverage for #358#359
laurentketterle-hub wants to merge 2 commits into
sourcey:mainfrom
laurentketterle-hub:boost-docs-ci-358

Conversation

@laurentketterle-hub

Copy link
Copy Markdown

Documentation & CI Boost

Complementing #358.

  • Enhanced README
  • CI workflow

@laurentketterle-hub
laurentketterle-hub force-pushed the boost-docs-ci-358 branch 2 times, most recently from a8c03b7 to bad7714 Compare August 9, 2026 23:14
@laurentketterle-hub

Copy link
Copy Markdown
Author

/claim 0x954dB727f224dAabe4F87E799843C2bB62dfd26C

@laurentketterle-hub

Copy link
Copy Markdown
Author

/claim 0x954dB727f224dAabe4F87E799843C2bB62dfd26C

@auscaster auscaster left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for this. A data pull request in this repository is data-only: it changes files under vendors/ and nothing else. This one also adds workflow and documentation files, which sourcey/validation rejects:

Catalog data changes cannot be mixed with other repository paths: .github/workflows/ci-boost.yml, README.md.

Drop the non-vendor files from the branch. Workflows and CI are owned by the Sourcey workspace and are not contributor-authored here.

@auscaster auscaster left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This pull request changes no vendor record, so there is nothing for the catalog round to review. The round accepts exactly one new vendor YAML under vendors/{shard}/, with no code, workflow, or documentation changes.

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.

3 participants