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

move switcher.json back to docs/bokeh #13903

Merged
merged 1 commit into from
May 21, 2024

Conversation

mosc9575
Copy link
Contributor

@mosc9575 mosc9575 commented May 18, 2024

This PR moves the file switcher.json back to docs/bokeh which should reenable the release build for docs.

The path ../switcher.json is working in conf.py, too, and the behavoir is the same as in #13901.

Edit: Maybe we can squash the latest two commits to reduce the changes in the git history after this is accepted and merged.

@mosc9575 mosc9575 added this to the 3.5 milestone May 18, 2024
@@ -55,9 +55,9 @@ def root():
return redirect("en/latest/index.html")


@app.route("/switcher.json")
@app.route("/en/switcher.json")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Combining en/latest/ and ../switcher.json results in /en/switcher.json for route.

Copy link

codecov bot commented May 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.15%. Comparing base (e61219b) to head (2dcce07).
Report is 41 commits behind head on branch-3.5.

Additional details and impacted files
@@              Coverage Diff               @@
##           branch-3.5   #13903      +/-   ##
==============================================
+ Coverage       91.57%   93.15%   +1.58%     
==============================================
  Files             326      282      -44     
  Lines           20737    20031     -706     
==============================================
- Hits            18990    18660     -330     
+ Misses           1747     1371     -376     

Copy link
Contributor

@mattpap mattpap left a comment

Choose a reason for hiding this comment

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

I will merge and see if this helps with 3.5.0.dev4 build.

@mattpap mattpap merged commit b402851 into bokeh:branch-3.5 May 21, 2024
26 of 27 checks passed
@mosc9575 mosc9575 deleted the 13902_fix_release_build branch May 27, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Release build fails after PR #13901
2 participants