Skip to content

Fix documentation build by replacing removed --polyglot flag #7292

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
afarber wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix documentation build by replacing removed --polyglot flag #7292

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f088228
Fix documentation build by replacing removed --polyglot flag with JSD…
afarber Dec 6, 2025
9cbb117
Add hint messages to error_on_dirty.sh for clearer CI failure output
afarber Dec 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions .github/workflows/osrm-backend.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -115,9 +115,11 @@ jobs:
- name: Run checks
run: |
./scripts/check_taginfo.py taginfo.json profiles/car.lua
./scripts/format.sh && ./scripts/error_on_dirty.sh
# Check that code formatting is up to date
./scripts/format.sh && ./scripts/error_on_dirty.sh "Run './scripts/format.sh' locally and commit the changes."
node ./scripts/validate_changelog.js
npm run docs && ./scripts/error_on_dirty.sh
# Check that API documentation is up to date with JSDoc comments in node_osrm.cpp
npm run docs && ./scripts/error_on_dirty.sh "Run 'npm run docs' locally and commit the changes."
npm audit --production

docker-image-matrix:
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
- Build:
- FIXED: Reduce MSVC compiler warnings by suppressing informational warnings while preserving bug-indicating warnings [#7253](https://github.com/Project-OSRM/osrm-backend/issues/7253)
- Misc:
- FIXED: Replace removed --polyglot flag with JSDoc extraction script for documentation.js v14 [#7274](https://github.com/Project-OSRM/osrm-backend/issues/7274)
- CHANGED: Add std::format compatibility layer with fallback to fmt::format [#7261](https://github.com/Project-OSRM/osrm-backend/pull/7261)
- FIXED: Update node_osrm to C++20 to fix ABI mismatch with libosrm (was overlooked in #6877) [#7261](https://github.com/Project-OSRM/osrm-backend/pull/7261)
- CHANGED: Update fmt library to version 11.2.0 [#7238](https://github.com/Project-OSRM/osrm-backend/issues/7238)
Expand Down
Loading
Loading