Skip to content

[HOLD] Revert "Fix anchor renderer for links in report messages & clean up"#10433

Closed
marcaaron wants to merge 2 commits intomainfrom
revert-10144-fix-links
Closed

[HOLD] Revert "Fix anchor renderer for links in report messages & clean up"#10433
marcaaron wants to merge 2 commits intomainfrom
revert-10144-fix-links

Conversation

@marcaaron
Copy link
Copy Markdown
Contributor

@marcaaron marcaaron commented Aug 17, 2022

Reverts #10144

Fixes:

$ #10431

@marcaaron marcaaron marked this pull request as ready for review August 17, 2022 19:22
@marcaaron marcaaron requested a review from a team as a code owner August 17, 2022 19:22
@melvin-bot melvin-bot bot requested review from sketchydroide and removed request for a team August 17, 2022 19:22
@francoisl
Copy link
Copy Markdown
Contributor

I restarted the failed tests, let's see if it helps 🤞

Also we'd want to get that CP'ed to staging, correct?

@github-actions
Copy link
Copy Markdown
Contributor

⚠️ ⚠️ Heads up! This pull request has the CP Staging label. ⚠️ ⚠️
Merging it will cause it to be immediately deployed to staging, even if the open StagingDeployCash deploy checklist is locked.

@marcaaron
Copy link
Copy Markdown
Contributor Author

Passing but @roryabraham says this will break stuff so maybe we hold off?

@francoisl francoisl changed the title Revert "Fix anchor renderer for links in report messages & clean up" [HOLD] Revert "Fix anchor renderer for links in report messages & clean up" Aug 18, 2022
@francoisl
Copy link
Copy Markdown
Contributor

Yeah I'm not 100% clear on the suggested way to go either, let's wait until he's back.

@roryabraham
Copy link
Copy Markdown
Contributor

Sorry, the main thing that's important is that if we CP a revert, we have to wait until the next release cycle before merging this same change back into main.

@marcaaron
Copy link
Copy Markdown
Contributor Author

I think this revert is no longer needed as we chose to deploy the broken code to production because it wasn't blocker-y enough.

@marcaaron marcaaron closed this Aug 24, 2022
@roryabraham roryabraham deleted the revert-10144-fix-links branch November 2, 2022 23:08
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.

3 participants