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

PA-12843 Resolve conflicts with report being overriden #98

Merged
merged 2 commits into from
Feb 1, 2024
Merged

PA-12843 Resolve conflicts with report being overriden #98

merged 2 commits into from
Feb 1, 2024

Conversation

SOOS-JAlvarez
Copy link
Contributor

@SOOS-JAlvarez SOOS-JAlvarez commented Feb 1, 2024 •
edited
Loading

Changes:

  • Rename traditional-json-headers to traditional-json just as it is on zapz
  • Removed extra cp operations and dir creations
  • Moved the cp operation to the zap hook and only use the report.json contents, template.yml is not needed.

Ticket: https://soos.atlassian.net/browse/PA-12843

@SOOS-JAlvarez SOOS-JAlvarez merged commit 0cbfad5 into main Feb 1, 2024
4 checks passed
@SOOS-JAlvarez SOOS-JAlvarez deleted the fix/PA-12843 branch February 1, 2024 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SOOS-GSteen SOOS-GSteen SOOS-GSteen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SOOS-JAlvarez @SOOS-GSteen