Skip to content
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.

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

[docs] use -W for sphinx mdx build #27949

Merged
merged 5 commits into from
Feb 20, 2025
Merged

Conversation

cmpadden
Copy link
Contributor

Summary & Motivation

Resolves issue where some users reported the following error message occurs when LC_ALL is not set.

locale_dir docs/sphinx/locales/en/LC_MESSAGES does not exist

How I Tested These Changes

yarn build-api-docs

Changelog

NOCHANGELOG

@cmpadden cmpadden requested a review from neverett as a code owner February 20, 2025 17:26
@cmpadden cmpadden changed the title [docs] set LC_ALL when building API docs locally [docs] use -W for sphinx mdx build Feb 20, 2025
@neverett
Copy link
Contributor

Tested locally and confirm this now works as expected (produces API docs and objects.inv file).

Copy link
Contributor Author

cmpadden commented Feb 20, 2025

Merge activity

  • Feb 20, 3:32 PM EST: A user started a stack merge that includes this pull request via Graphite.
  • Feb 20, 3:33 PM EST: A user merged this pull request with Graphite.

@cmpadden cmpadden merged commit 3475388 into master Feb 20, 2025
6 of 7 checks passed
@cmpadden cmpadden deleted the colton/lc-all-build-script branch February 20, 2025 20:33
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