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.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Syncing upstream helm charts #15

Closed
wants to merge 2 commits into from
Closed

Syncing upstream helm charts #15

wants to merge 2 commits into from

Conversation

jkremser
Copy link
Member

馃摝 Syncing upstream helm chart 馃摝

:shipit: Release?

The sync mechanism doesn't deal with releases.

馃 Rebase or merge?

Although merging is obviously better :trollface:, let's use rebasing for this sync.
This way we will see our custom tweaks nicely on the tip of the main.

馃彞 Conflict?

This could have happened only if the PR was opened for a long time and meanwhile there
was a change to this repo. 馃┕ Just close this PR and wait for the next one.

馃挕 Tip

Ideally, this repo has only one commits with all the necessary tweaks in it (like this gh action for instance).
So that we can keep adding the changes to this commit by g commit --ammend and ideally the changes should be in
new files so that the risk of conflict with the upstream repo is minimal.

馃敡 Important

Once this is merged, please don't forget to run this command if our 'All our tweaks should go here' commit is not on
top:

GIT_SEQUENCE_EDITOR=\"sed -i -n 'h;1n;2p;g;p'\" git rebase -i HEAD~2

To reorder last two commits.

This automated PR was created by this action.

JorTurFer and others added 2 commits April 25, 2024 15:37
* Packaged new Helm chart version

Signed-off-by: Jorge Turrado <jorge.turrado@scrm.lidl>

* fix typo

Signed-off-by: Jorge Turrado <jorge.turrado@scrm.lidl>

---------

Signed-off-by: Jorge Turrado <jorge.turrado@scrm.lidl>
If there is another thing to add to our fork, pls add it here.

Example:

```
echo -e "#foo?\n\n bar?" > foo-bar.md # Pls keep adding the changes to standalone files that are not present in the upstream to mitigate the risk of conflict
add foo-bar.md
git commit -s --ammend
yada yada
```

Signed-off-by: Jirka Kremser <jiri.kremser@gmail.com>
@jkremser jkremser closed this Apr 28, 2024
@jkremser jkremser deleted the upstream-sync branch April 28, 2024 07:04
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.

None yet

2 participants