Skip to content

ci: consolidate language workflows and unify runtime versions - #1468

Merged
tushuhei merged 3 commits into
mainfrom
ci/consolidate-workflows
Oct 26, 2026
Merged

tushuhei merged 3 commits into
mainfrom
ci/consolidate-workflows

Conversation

@tushuhei

Copy link
Copy Markdown
Member

Summary

  • Consolidate per-language style checks and unit tests into single-job workflows (python.yml, javascript.yml, java.yml) so each language sets up its runtime and dependencies once in a single place, preventing version drift between lint and test jobs.
  • Add "engines": {"node": ">=22"} to javascript/package.json and demo/package.json, and configure build-demo.yml to read node-version-file: 'javascript/package.json' instead of hardcoding Node 20.
  • Unify Java version to JDK 21 across style check and unit test steps in java.yml.
  • Narrow style-check.yml to repository-wide Markdown linting (**/*.md).
  • Fix trigger path gaps so changes to shared model files (budoux/models/**, budoux/skip_nodes.json) or workflow files trigger JavaScript, Java, and Demo CI, and validate demo builds on pull requests.
  • Add dependency caching (uv, npm, maven), concurrency cancellation, and explicit timeout-minutes across all workflows.

- Merge per-language style checks and unit tests into single-job workflows (python.yml, javascript.yml, java.yml) to eliminate duplicated setup and runtime version drift.
- Add engines.node (>=22) to javascript/package.json and demo/package.json and use node-version-file in build-demo.yml.
- Add missing path triggers for shared model data (budoux/models/**, budoux/skip_nodes.json) and PR build verification for demo.
- Enable dependency caching (uv, npm, maven), concurrency cancellation, and explicit job timeouts across all workflows.

Change-Id: Iebffffeee2bb5934551414dcf15d7a6692663133
@tushuhei
tushuhei requested a review from kojiishi October 26, 2026 01:23
- Replace nosborn/github-action-markdown-cli with DavidAnson/markdownlint-cli2-action (v24.2.0, Node 24 native).
- Fix MD013, MD022, and MD031 violations in README.md and javascript/README.md that were previously missed because nosborn/github-action-markdown-cli expanded unquoted '**/*.md' via /bin/sh (skipping root-level Markdown files) and used an older markdown-it HTML block parser.

Change-Id: I09acc69dab0616ce2f45c859f78ca9ceb8b2edb1
Change-Id: I292b1cb36f8a6751f8f46c0e814b53dd62dde4e9

@kojiishi kojiishi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@tushuhei
tushuhei merged commit fc2e8c6 into main Oct 26, 2026
41 checks passed
@tushuhei
tushuhei deleted the ci/consolidate-workflows branch October 26, 2026 10:48
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