-
Notifications
You must be signed in to change notification settings - Fork 121
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
feat: adding message with link to the release when releasing an existing release #1891
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1891 +/- ##
============================================
- Coverage 100.00% 87.93% -12.07%
============================================
Files 173 174 +1
Lines 5446 5454 +8
============================================
- Hits 5446 4796 -650
- Misses 0 658 +658
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
The links aren't secret, btw. :) You don't need to scratch yours out. We separately have a bug about doing a better job of hiding potentially sensitive information in our logs: #1770 |
|
||
void main() { | ||
group('ShorebirdWebConsole', () { | ||
test('linkTo returns the expected link to the root of the site', () { |
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.
test('linkTo returns the expected link to the root of the site', () { | |
test('linkTo returns the correct link relative to the root of the console', () { |
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 with felix's link change.
packages/shorebird_cli/test/src/shorebird_web_console_test.dart
Outdated
Show resolved
Hide resolved
packages/shorebird_cli/test/src/code_push_client_wrapper_test.dart
Outdated
Show resolved
Hide resolved
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.
Left a couple nits, but LGTM otherwise!
packages/shorebird_cli/test/src/shorebird_web_console_test.dart
Outdated
Show resolved
Hide resolved
Co-authored-by: Felix Angelov <felix@shorebird.dev>
Description
Adds a new message to the shorebird release [any] that will print a link to the release on the console when trying to run release in a version number that already exists!
Fixes #1890
Output
Type of Change