-
Notifications
You must be signed in to change notification settings - Fork 212
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
Add redirects for our project boards #926
Conversation
The URLs are kind of long and these will help us out if we ever need to change them in the future.
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.
Oooh good call!
Should we update the links in the README.md in this PR or a follow-up? |
Oh wait I just found a few more redirects, for good first issue and help wanted that I think would be really useful as well. I'll make those and update the docs to use them. |
@vdice PTAL |
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.
LGTM!
/board https://github.com/orgs/deislabs/projects/2 | ||
/board/* https://github.com/orgs/deislabs/projects/2?card_filter_query=label:":splat" | ||
/roadmap https://github.com/deislabs/porter/projects/4 | ||
/src/* https://github.com/deislabs/porter/blob/master/:splat |
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.
👍
What does this change
The URLs are kind of long and these will help us out if we ever need to change them in the future.
/src/PATH
, e.g./src/CONTRIBUTING.md
What issue does it fix
Me typing too much
Notes for the reviewer
We can test these url's out on the netlify preview
Checklist