-
Notifications
You must be signed in to change notification settings - Fork 162
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
Add big-bang-umbrella
folder back to Zarf examples...
#308
Add big-bang-umbrella
folder back to Zarf examples...
#308
Conversation
…to-new-software-factory-repo
/test all |
/test all |
…redirects-to-new-software-factory-repo' into feature/add-example-folder-that-redirects-to-new-software-factory-repo
/test all |
/test all |
/test all |
/test all |
Ready for review & merge |
@RothAndrew I'm confused on this PR a little bit, I see the readme updates and makes total sense, but "...also fix broken E2E test" seems completely unrelated to the topic PR. Now of course I don't mind combining these as I do it all the time, but think the PR topic needs to be updated because a README change and E2E tests are very different, very unrelated things. Thoughts? |
I can pull the test change out into its own PR. I definitely want to practice what I preach 😄 |
…to-new-software-factory-repo
/test all |
@jeff-mccoy ready for another review |
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.
🦄
...with a README that redirects the reader to the new repo that replaced it
...with a README that redirects the reader to the new repo that replaced it