-
Notifications
You must be signed in to change notification settings - Fork 30
FW-56 Add skynet.yaml #63
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
Conversation
Security InsightsNo security relevant content was detected by automated scans. Action Items
Questions or Comments? Reach out on Slack: #support-infosec. |
run_tests.sh
Outdated
pub get | ||
|
||
pub run test --file-reporter=json:$filename.json || RESULT=1 |
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.
Would you mind changing these to use the new CLI? The top-level pub
CLI is now deprecated in favor of dart pub
, and pub run test
specifically is replaced by dart test
:
pub get | |
pub run test --file-reporter=json:$filename.json || RESULT=1 | |
dart pub get | |
dart test --file-reporter=json:$filename.json || RESULT=1 |
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.
My bad of course, updated.
QA +1
|
@Workiva/release-management-p |
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.
+1 from RM
This runs the unit tests inside skynet to satisfy the release pipeline QA requirement.