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

Move index branch workflows to main #25730

Merged
merged 2 commits into from May 15, 2024
Merged

Move index branch workflows to main #25730

merged 2 commits into from May 15, 2024

Conversation

jotamartos
Copy link
Contributor

@jotamartos jotamartos commented May 13, 2024 •
edited

Description of the change

Moving all workflows to main branch

Benefits

All workflows are located together

Possible drawbacks

None AFAIK

Applicable issues

Additional information

Related to #25712

Checklist

  • [Na] Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • [NA] Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • [NA] Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

@jotamartos
Reduce archive-full-index branch size by squashing all commits
8efc649 
Signed-off-by: Jota Martos <jotamartos@vmware.com>
@carrodher carrodher mentioned this pull request May 13, 2024
Merged
1 task
@jotamartos
Remove index branch workflows after moving them to main
357dd9c 
Signed-off-by: Jota Martos <jotamartos@vmware.com>
@jotamartos jotamartos changed the title Reduce archive-full-index branch size by squashing all commits Move index branch workflows to main May 14, 2024
@jotamartos jotamartos merged commit 7d6426d into index May 15, 2024
1 check passed
@jotamartos jotamartos deleted the reduce-size-branches-index branch May 15, 2024 07:54
fmulero
fmulero reviewed May 15, 2024
View reviewed changes
Copy link
Collaborator

@fmulero fmulero left a comment

Choose a reason for hiding this comment

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

LGTM

@jotamartos
Copy link
Contributor Author

We needed to recover these changes later

#25883
#25884

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fmulero fmulero fmulero left review comments

@carrodher carrodher carrodher approved these changes

@javsalgar javsalgar Awaiting requested review from javsalgar

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

Successfully merging this pull request may close these issues.

None yet
3 participants
@jotamartos @carrodher @fmulero