-
-
Notifications
You must be signed in to change notification settings - Fork 620
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(traefik): Add stripPrefixRegex #2422
feat(traefik): Add stripPrefixRegex #2422
Conversation
Signed-off-by: Stavros Ntentos <133706+stdedos@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm missing the required minor version bump in Chart.yaml and the definition + comments in values.yaml
> I'm missing the required minor version bump in Chart.yaml and the definition + comments in values.yaml Additionally, add the changelog entry to-be Signed-off-by: Stavros Ntentos <133706+stdedos@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see above comments
Signed-off-by: Stavros Ntentos <133706+stdedos@users.noreply.github.com>
Oops 😓 Hopefully it's clean now 🙏 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me! :)
Thank you for your submission, may users inter-alia requested this.
Your Helm templating skills are quite good though :)
@all-contributors please add @stdedos for code
@all-contributors please add @stdedos for code |
I've put up a pull request to add @stdedos! 🎉 |
It took me around 6h-8h bootstrapping (traefik, repo, instructions, piecing together etc), I am glad it was that easy at the end to merge!
No, my investigation skills are good enough; my goofness too 😛 e.g. I have no idea why/how I missed your first comment; nor "what does it mean" anyway.
Any "plans" when is this going live? |
I've updated the pull request to add @stdedos! 🎉 |
Well, I rather have fine copy-pasters than mediocre coders ;-) New version should be online in a few minutes or so :) |
This PR is locked to prevent necro-posting on closed PRs. Please create a issue or contact staff on discord if you want to further discuss this |
Signed-off-by: Stavros Ntentos 133706+stdedos@users.noreply.github.com
Description
⚒️ Fixes #
⚙️ Type of change
🧪 How Has This Been Tested?
It hasn't, but then again I was instructed to just open a PR for it https://discord.com/channels/830763548678291466/886351979306188811/960460336572551168
📃 Notes:
✔️ Checklist: