-
Notifications
You must be signed in to change notification settings - Fork 128
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
Update release versions redirects for CL #928
Conversation
This also updates the redirect for the prefix `/about/` to use OLS instead of Ontobee to be aligned with the term browser.
config/cl.yml
Outdated
@@ -68,8 +68,35 @@ entries: | |||
- prefix: /2011-12-13/ | |||
replacement: https://cell-ontology.googlecode.com/svn/releases/2011-12-13/ | |||
|
|||
- regex: ^/releases/(201.*)/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't use the regex
system very often. The examples in the README all end with $
but I don't think that's what you want here.
I think you could get what you want with a series of prefix
rules.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, it's conflicting with the new rule for release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is still failing the tests.
Entries are processed in order from top to bottom, which is why I think that a series of prefix
entries will do what you want.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will it work to only change the key regex
to prefix
?
Thank you, James! 🙏🏼 |
I think this is what you want. It passes all the tests you wrote. Good to merge? |
Yes, thank you! |
This also updates the redirect for the prefix
/about/
to use OLS instead of Ontobee to be aligned with the term browser.Related to obophenotype/cell-ontology#2048