Skip to content

Patch pipelines release v0.2 once more#1415

Merged
simonbaird merged 3 commits into
conforma:release-v0.2from
simonbaird:patch-pipelines-release-v0.2-once-more
Mar 8, 2024
Merged

Patch pipelines release v0.2 once more#1415
simonbaird merged 3 commits into
conforma:release-v0.2from
simonbaird:patch-pipelines-release-v0.2-once-more

Conversation

@simonbaird

Copy link
Copy Markdown
Member

This is the same as PR #1411 but hopefully with the correct target branch. 😅 .

red-hat-konflux and others added 2 commits March 8, 2024 09:23
The original version was in PR conforma#1391, but I cherry picked this
commit while re-doing the release-v0.2 branch, which I'm planning to
force push shortly.

Ref: https://issues.redhat.com/browse/EC-434
Apply changes to the Konflux generated default pipelines.

Also remove the main branch pipelines since they're not needed in the release branch.

(Commit created with hack/patch-release-pipelines.sh )
@@ -10,10 +10,10 @@ metadata:
== "main"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should be release-v0.2

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good catch, thanks.

Copy the pipeline definitions from the main branch versions and apply name changes.

Also remove the main branch pipelines since they're not needed in the release branch.

(Commit created with hack/copy-task-release-pipelines.sh)
@simonbaird simonbaird force-pushed the patch-pipelines-release-v0.2-once-more branch from f0b8876 to d87616f Compare March 8, 2024 17:18
@codecov

codecov Bot commented Mar 8, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.82%. Comparing base (cec8190) to head (d87616f).

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##           release-v0.2    #1415      +/-   ##
================================================
+ Coverage         79.59%   86.82%   +7.22%     
================================================
  Files                62       73      +11     
  Lines              4347     4682     +335     
================================================
+ Hits               3460     4065     +605     
+ Misses              887      617     -270     
Flag Coverage Δ
acceptance 72.07% <ø> (?)
generative 79.59% <ø> (ø)
integration 79.59% <ø> (ø)
unit 79.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 39 files with indirect coverage changes

value: Dockerfile.dist
- name: git-url
value: '{{repo_url}}'
value: '{{source_url}}'

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I've got another commit to fix this in main branch..

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@simonbaird simonbaird merged commit 29da61a into conforma:release-v0.2 Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants