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.

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

Correct previous bad merge #4800

Merged
merged 1 commit into from
Mar 5, 2021
Merged

Correct previous bad merge #4800

merged 1 commit into from
Mar 5, 2021

Conversation

msorens
Copy link
Contributor

@msorens msorens commented Mar 4, 2021

🔩 Description: What code changed, and why?

The merge of PR #4425 (commit 21371a8)
re-materialized code that was deleted in the prior commit (84333a1).
This deletes that code... again.
(FYI: It has migrated over to verify_private.pipeline.yml).

⛓️ Related Resources

Noted above.

👍 Definition of Done

Semgrep no longer appears on https://buildkite.com/chef-oss/chef-automate-master-verify

👟 How to Build and Test the Change

Just check the buildkite page above.

✅ Checklist

📷 Screenshots, if applicable

NA

The merge of PR #4425 (commit 21371a8)
re-materialized code that was deleted in the prior commit (84333a1).
This deletes that code... again.
(FYI: It has migrated over to verify_private.pipeline.yml).

Signed-off-by: michael sorens <[email protected]>
@msorens msorens self-assigned this Mar 4, 2021
@netlify
Copy link

netlify bot commented Mar 4, 2021

Deploy preview for chef-automate processing.

Building with commit 1ff347c

https://app.netlify.com/sites/chef-automate/deploys/604172c635fc87000765f56b

Copy link
Contributor

@danielsdeleo danielsdeleo left a comment

Choose a reason for hiding this comment

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

Yay for fresh salmon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants