sadpandajoe opened a new pull request, #35185:
URL: https://github.com/apache/superset/pull/35185

   <!---
   Please write the PR title following the conventions at 
https://www.conventionalcommits.org/en/v1.0.0/
   Example:
   fix(dashboard): load charts correctly
   -->
   
   ### SUMMARY
   <!--- Describe the change below, including rationale and design decisions -->
   Fixed MDX compilation errors in documentation build by escaping comparison 
operators (`<=`, `>=`) in version strings. The root cause was MDX interpreting 
these operators as JSX syntax rather than plain text, causing CI build failures.
   
   **Root Cause:** MDX parser was attempting to parse comparison operators as 
JSX expressions when they appeared outside of code blocks, resulting in syntax 
errors like "Unexpected character `=` (U+003D) before name".
   
   **Solution:** Wrapped version strings containing comparison operators in 
backticks to treat them as inline code, preventing MDX from attempting JSX 
parsing.
   
     **Files affected:**
     - `docs/docs/configuration/alerts-reports.mdx` - Fixed `<=4.0.x` and 
`>=4.1.x`
     - `docs/docs/configuration/theming.mdx` - Fixed `apache-superset>=6.0`
     - `docs/docs/contributing/guidelines.mdx` - Fixed `>=30 days`
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   <!--- Skip this if not applicable -->
   
   ### TESTING INSTRUCTIONS
   <!--- Required! What steps can be taken to manually verify the changes? -->
     1. Run documentation build: `cd docs && yarn build`
     2. Verify build completes without MDX compilation errors
     3. Check that version strings display correctly as inline code in the 
rendered documentation
   
   
   ### ADDITIONAL INFORMATION
   <!--- Check any relevant boxes with "x" -->
   <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->
   - [ ] Has associated issue:
   - [ ] Required feature flags:
   - [ ] Changes UI
   - [ ] Includes DB Migration (follow approval process in 
[SIP-59](https://github.com/apache/superset/issues/13351))
     - [ ] Migration is atomic, supports rollback & is backwards-compatible
     - [ ] Confirm DB migration upgrade and downgrade tested
     - [ ] Runtime estimates and downtime expectations provided
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org

Reply via email to