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

Remove inventory redirect #67

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from

Conversation

tacaswell
Copy link
Member

Per discussion in
matplotlib/matplotlib#22601 (comment)
this is actually breaking builds in subtle very brittle way as the links that
will be generated will be pointing at the top-level files (that are actually
re-directs to /stable).

The second step of this will be to put a 301 redirect into caddy.

Per discussion in
matplotlib/matplotlib#22601 (comment)
this is actually breaking builds in subtle very brittle way as the links that
will be generated will be pointing at the top-level files (that are actually
re-directs to /stable).

The second step of this will be to put a 301 redirect into caddy.
@QuLogic
Copy link
Member

QuLogic commented Mar 10, 2022

We're actually serving from main there, though we could switch to gh-pages.

@tacaswell tacaswell changed the base branch from gh-pages to main March 10, 2022 22:52
@tacaswell
Copy link
Member Author

Changed the bass branch. From reading the ansible (~L95) where we do the checkouts all of the repos go to gh-pages so I thought we were doing gh-pages everywhere.

@QuLogic
Copy link
Member

QuLogic commented Mar 10, 2022

Yea, they're currently in sync, so I was a bit lazy with writing the Ansible, but that's not what's currently being served.

@tacaswell
Copy link
Member Author

Given that the caddy configuration seems to be taking precedence anyway, this is a pretty low priority to get done (mostly to avoid confusing future us).

@tacaswell tacaswell changed the base branch from main to gh-pages April 18, 2022 15:07
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