-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fix cicd install bundler #1330
Fix cicd install bundler #1330
Conversation
114c43b
to
9f21ca5
Compare
…tlementcard into fix-ci-bundle-install
270b69c
to
c84f926
Compare
995532b
to
017f2ec
Compare
…tlementcard into fix-ci-bundle-install
…tlementcard into fix-ci-bundle-install
…tlementcard into fix-ci-bundle-install
bb98b90
to
ac147ba
Compare
ac147ba
to
ece3b90
Compare
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 :)
Just one comment
|
||
- run: | ||
command: | | ||
app-toolbelt v0 notify mattermost --message "<< parameters.success_message >>" --channel << parameters.channel >> <<#parameters.allow-all-branches>>--allow-all-branches<</parameters.allow-all-branches>> |
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.
is this "#" correct here? #parameters.allow-all-branches
Sorry could mark the correct line. its in the next line
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.
yes it is correct, this is some circleci magic syntax to add the content between the both << >>
if the parameter is true.
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.
yes i know. but i'm talking about the additional #
what does that mean.
Update: ah is this an 'open' tag?
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.
👍 nice work :) thx for fixing
Short description
delivery is failing due to wrong permissions while installing the gems, see here: https://app.circleci.com/pipelines/github/digitalfabrik/entitlementcard/4572/workflows/75290bc2-3332-49b1-8762-74c3ea5ab9e9/jobs/35497
Proposed changes