Skip to content

Remove unused parameters and result#737

Closed
mkosiarc wants to merge 0 commit into
konflux-ci:mainfrom
mkosiarc:remove-skip-optional
Closed

Remove unused parameters and result#737
mkosiarc wants to merge 0 commit into
konflux-ci:mainfrom
mkosiarc:remove-skip-optional

Conversation

@mkosiarc
Copy link
Copy Markdown
Contributor

@mkosiarc mkosiarc commented Jan 11, 2024

The parameters skip-optional, pipelinerun-name and pipelinerun-uid are
no longer used in init task and neither is the container-registry-secret
result.
New version for init task was created together with the migration doc
and the reference was updated in the pipeline

STONEBLD-2000

@mkosiarc
Copy link
Copy Markdown
Contributor Author

/retest

@mkosiarc mkosiarc force-pushed the remove-skip-optional branch 2 times, most recently from 53a53d7 to 5063b46 Compare January 11, 2024 10:42
@mkosiarc
Copy link
Copy Markdown
Contributor Author

/retest

Copy link
Copy Markdown
Member

@mmorhun mmorhun left a comment

Choose a reason for hiding this comment

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

LGTM

@mkosiarc mkosiarc force-pushed the remove-skip-optional branch from 5063b46 to 183f396 Compare January 12, 2024 08:10
@mkosiarc
Copy link
Copy Markdown
Contributor Author

/retest

1 similar comment
@mkosiarc
Copy link
Copy Markdown
Contributor Author

/retest

Comment thread task/init/0.2/README.md Outdated
Comment thread task/init/0.2/MIGRATION.md Outdated
@mkosiarc mkosiarc force-pushed the remove-skip-optional branch from 183f396 to 7ed5bff Compare January 15, 2024 12:21
@mkosiarc mkosiarc changed the title Remove skip-optional parameter Remove unused parameters and result Jan 15, 2024
@mkosiarc mkosiarc force-pushed the remove-skip-optional branch from 7ed5bff to 6e59b81 Compare January 15, 2024 12:23
Copy link
Copy Markdown
Contributor

@chmeliik chmeliik left a comment

Choose a reason for hiding this comment

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

LGTM, tiny nitpick in the migration doc

Comment thread task/init/0.2/MIGRATION.md Outdated
@mkosiarc mkosiarc force-pushed the remove-skip-optional branch from 6e59b81 to ac6912f Compare January 15, 2024 12:52
@mkosiarc
Copy link
Copy Markdown
Contributor Author

/retest

4 similar comments
@mkosiarc
Copy link
Copy Markdown
Contributor Author

/retest

@mkosiarc
Copy link
Copy Markdown
Contributor Author

/retest

@mkosiarc
Copy link
Copy Markdown
Contributor Author

/retest

@mkosiarc
Copy link
Copy Markdown
Contributor Author

/retest

@mkosiarc mkosiarc closed this Jan 16, 2024
@mkosiarc mkosiarc force-pushed the remove-skip-optional branch from ac6912f to 33e86b9 Compare January 16, 2024 10:52
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@mkosiarc
Copy link
Copy Markdown
Contributor Author

mkosiarc commented Jan 16, 2024

Accidentally closed this PR by pushing main branch, could not reopen - opened a new identical PR #746

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.

4 participants