Skip to content

Rebuilding CI tests#253

Merged
jiashuy merged 1 commit into
NVIDIA-Merlin:masterfrom
jiashuy:master
May 22, 2026
Merged

Rebuilding CI tests#253
jiashuy merged 1 commit into
NVIDIA-Merlin:masterfrom
jiashuy:master

Conversation

@jiashuy

@jiashuy jiashuy commented May 21, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@jiashuy jiashuy changed the title Placeholder PR for building CI tests env [Draft] Placeholder PR for building CI tests env May 21, 2026
@jiashuy jiashuy changed the title [Draft] Placeholder PR for building CI tests env Rebuilding CI tests May 21, 2026
- .github/workflows/blossom-ci.yml: switch the trigger match to
  startsWith() so '/blossom-ci args' forms are accepted (the strict
  '==' was rejecting any extra payload), bump actions/checkout to v4,
  and clean up stray trailing whitespace. Allowlist and trigger word
  are unchanged.
- .github/workflows/lint.yml: new public PR check that runs
  clang-format --dry-run --Werror over include/, tests/ and benchmark/
  on free ubuntu-22.04 runners. No GPU needed, lands feedback within a
  couple of minutes of PR open.
- .pre-commit-config.yaml: matches what lint.yml runs (clang-format)
  plus codespell. Contributors can 'pre-commit install' locally to
  catch the same issues before pushing.
@jiashuy

jiashuy commented May 21, 2026

Copy link
Copy Markdown
Collaborator Author

/blossom-ci

@rhdong rhdong 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.

LGTM, thanks!

@jiashuy jiashuy merged commit 9fa33c6 into NVIDIA-Merlin:master May 22, 2026
0 of 3 checks passed
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.

2 participants