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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor!: rename APP_HOST into PIPELINE_CALLBACK_URL #403

Merged
merged 5 commits into from
Sep 13, 2023

Conversation

shiipou
Copy link
Contributor

@shiipou shiipou commented Aug 14, 2023

Closes #

Description of the changes

Rename the APP_HOST env var that the name didn't really mean what the variable do in the code.
I renamed it PIPELINE_CALLBACK_URL because it's the callback the runner will call at the end of each builds to update the build status.

Checklist

  • I didn't over-scope my PR
  • My PR title matches the commit convention
  • I did not include breaking changes
  • I made my own code-review before requesting one

I included unit tests that cover my changes

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

I added/updated the documentation about my changes

  • 📜 README.md
  • 📕 docs/*.md
  • 📓 docs.lenra.io
  • 🙅 no documentation needed

Technical highlight/advice

Signed-off-by: shiipou <shiishii@nocturlab.fr>
Signed-off-by: shiipou <shiishii@nocturlab.fr>
@shiipou shiipou self-assigned this Aug 14, 2023
@taorepoara
Copy link
Member

You must enter you PR description.

@taorepoara taorepoara merged commit 97d10f6 into beta Sep 13, 2023
6 checks passed
@taorepoara taorepoara deleted the quick-fix-kubernetes-runner-never-succeed branch September 13, 2023 09:25
@taorepoara
Copy link
Member

🎉 This PR is included in version 1.0.0-beta.160 🎉

The release is available on:

Your semantic-release bot 📦🚀

@taorepoara
Copy link
Member

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants