Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add empty skynet plan so release pipelines will stop failing #745

Merged
merged 1 commit into from
May 10, 2022
Merged

Add empty skynet plan so release pipelines will stop failing #745

merged 1 commit into from
May 10, 2022

Conversation

aaronlademann-wf
Copy link
Contributor

self explanatory

@aaronlademann-wf
Add empty skynet plan
3144f65 
so release pipelines will stop failing
@aviary-wf
Copy link

Security Insights

No security relevant content was detected by automated scans.

Action Items

  • Review PR for security impact; comment "security review required" if needed or unsure
  • Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

KealJones
KealJones approved these changes May 10, 2022
View reviewed changes
Copy link

@KealJones KealJones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

skynet.yaml
image: drydock.workiva.net/workiva/skynet-images:node-test-image-latest

scripts:
- echo "I have made RM happy"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#nit

Suggested change
- echo "I have made RM happy"
- print "I have made RM happy"

im just kidding it literally makes 0 difference.

@aaronlademann-wf
Copy link
Contributor Author

@Workiva/release-management-pp

@aaronlademann-wf
Copy link
Contributor Author

@Workiva/release-management-pp

@KealJones
Copy link

KealJones commented May 10, 2022 •
edited
Loading

+1

KealJones
KealJones approved these changes May 10, 2022
View reviewed changes
Copy link

@KealJones KealJones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@KealJones
Copy link

+10

@KealJones
Copy link

@rmconsole-wf

kealjones-wk
kealjones-wk approved these changes May 10, 2022
View reviewed changes
Copy link
Contributor

@kealjones-wk kealjones-wk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1
@Workiva/release-management-p

rmconsole-wf
rmconsole-wf approved these changes May 10, 2022
View reviewed changes
Copy link

@rmconsole-wf rmconsole-wf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 from RM

@rm-astro-wf rm-astro-wf merged commit 3eee062 into master May 10, 2022
@rm-astro-wf rm-astro-wf deleted the add-skynet-plan branch May 10, 2022 21:24
@rmconsole-readonly-wk rmconsole-readonly-wk mentioned this pull request May 11, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KealJones KealJones KealJones approved these changes

@rmconsole-wf rmconsole-wf rmconsole-wf approved these changes

@kealjones-wk kealjones-wk kealjones-wk approved these changes

Assignees
No one assigned
Labels
Merge Requirements Met RM Ready
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@aaronlademann-wf @aviary-wf @KealJones @rmconsole-wf @kealjones-wk @rm-astro-wf