-
Notifications
You must be signed in to change notification settings - Fork 14k
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
fix: update standalone url for slack reports #18756
Conversation
Codecov Report
@@ Coverage Diff @@
## master #18756 +/- ##
=======================================
Coverage 66.31% 66.31%
=======================================
Files 1620 1620
Lines 63080 63082 +2
Branches 6370 6370
=======================================
+ Hits 41833 41835 +2
Misses 19591 19591
Partials 1656 1656
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Looks great! Thanks for jumping on this. Can you also write a test for it? I think you could maybe repurpose @AAfghahi's. |
Just updated slack test that give us access to the url |
2507822
to
903aa4e
Compare
🏷 preset:2022.7 |
* update url for slack reports * fix test * update text to be dynamic (cherry picked from commit 5d2e726)
SUMMARY
Following #18157 we forgot to update the slack url with the new modify url function
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION