-
-
Notifications
You must be signed in to change notification settings - Fork 186
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 links to open in new tab #617
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #617 +/- ##
=======================================
Coverage 81.66% 81.66%
=======================================
Files 33 33
Lines 4139 4139
=======================================
Hits 3380 3380
Misses 695 695
Partials 64 64 ☔ View full report in Codecov by Sentry. |
Hi, thanks for the PR! The links are generated here: Line 158 in e8f4568
You would have to change that, otherwise the CI would overwrite your change as soon as we merge it. |
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.
See comment before (#617 (comment))
Description
TemplatePrinter
wasTextPrinter
Scope
Related Issue
To-Do Checklist