-
Notifications
You must be signed in to change notification settings - Fork 918
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 builders.m.o location #15614
Update builders.m.o location #15614
Conversation
@janbrasna needs a rebase, sorry! |
fd7f5d1
to
5ea5040
Compare
This comment was marked as resolved.
This comment was marked as resolved.
@janbrasna The URL change will be handled initially by a redirect within future.m.o which will be pushed live at launch time. So feel free to remove the URL change from this PR if it makes things simpler |
5ea5040
to
b9da0e0
Compare
I understood the request the other way around; that until live the subdomain would redirect back to current future.m.o location. I've separated out the ftl change to #15624 so this can sit around until needed. |
The builders.m.o redirect in future.m.o is now live, FWIW |
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.
R+ 🏗️
@janbrasna Something went horribly wrong when I tried to fix a merge conflict 😫 Could you please rebase and force push? |
67a2291
to
8ec3b2c
Compare
@stephaniehobson NP, I had that ref already prepared & resolved, you were just quicker;) The merge did seem alright, but here's a clean rebase nonetheless, passing. ✅ |
One-line summary
Updates future.mozilla.org/builders link and relevant redirects to builders.mozilla.org
Significant changes and points to review
https://builders.mozilla.org is up now so this is good to go.
Issue / Bugzilla link
#15613
Testing
http://localhost:8000/builders/