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

extract common pull/push part in verification github actions #963

Merged
merged 2 commits into from
May 9, 2024
Merged

extract common pull/push part in verification github actions #963

merged 2 commits into from
May 9, 2024

Conversation

anoipm
Copy link
Contributor

@anoipm anoipm commented May 7, 2024 •
edited
Loading

Description

Changes proposed in this pull request:

  • split verification test gh workflows,
  • unify gh workflows, jobs and steps names,
  • extract collect-cluster-info gh action.

Related issue(s)
See also #961

@kyma-bot kyma-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 7, 2024
@kyma-bot
Copy link
Contributor

kyma-bot commented May 7, 2024

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@kyma-bot kyma-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cla: yes Indicates the PR's author has signed the CLA. labels May 7, 2024
@anoipm anoipm added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 8, 2024
@anoipm anoipm marked this pull request as ready for review May 8, 2024 10:26
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 8, 2024
@anoipm
split verification test gh workflows
247f062 
unify gh workflows, jobs and steps names
extract collect-cluster-info gh action
@anoipm anoipm added kind/feature Categorizes issue or PR as related to a new feature. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. area/serverless Issues or PRs related to serverless and removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels May 8, 2024
@anoipm anoipm mentioned this pull request May 8, 2024
Merged
pPrecel
pPrecel approved these changes May 9, 2024
View reviewed changes
Copy link
Contributor

@pPrecel pPrecel left a comment

Choose a reason for hiding this comment

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

Great Changes 👍🏿

@kyma-bot kyma-bot added the lgtm Looks good to me! label May 9, 2024
@kyma-bot kyma-bot merged commit 4177a0c into kyma-project:main May 9, 2024
18 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pPrecel pPrecel pPrecel approved these changes

@dbadura dbadura Awaiting requested review from dbadura dbadura is a code owner

@kwiatekus kwiatekus Awaiting requested review from kwiatekus kwiatekus is a code owner

@Cortey Cortey Awaiting requested review from Cortey Cortey is a code owner

@MichalKalke MichalKalke Awaiting requested review from MichalKalke MichalKalke is a code owner

@halamix2 halamix2 Awaiting requested review from halamix2 halamix2 is a code owner

Assignees

@pPrecel pPrecel

Labels
area/serverless Issues or PRs related to serverless cla: yes Indicates the PR's author has signed the CLA. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. kind/feature Categorizes issue or PR as related to a new feature. lgtm Looks good to me! size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@anoipm @kyma-bot @pPrecel