Skip to content

Add feature flags to the composite action#36

Merged
krlvi merged 2 commits intomainfrom
action-feature-flags
Jun 13, 2022
Merged

Add feature flags to the composite action#36
krlvi merged 2 commits intomainfrom
action-feature-flags

Conversation

@krlvi
Copy link
Member

@krlvi krlvi commented Jun 13, 2022

No description provided.

@krlvi krlvi force-pushed the action-feature-flags branch 2 times, most recently from 4fa21ad to 22c952d Compare June 13, 2022 12:43
@krlvi krlvi force-pushed the action-feature-flags branch 2 times, most recently from bb1e054 to 57c10ca Compare June 13, 2022 13:17
action.yml Outdated
# If Codeball didn't approve the contribution, fail the job.
- name: Fail Job
shell: bash
if: ${{ steps.codeball_status.outputs.approved == 'fail' && inputs.failJobsWhenReviewNeeded == 'true' }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
if: ${{ steps.codeball_status.outputs.approved == 'fail' && inputs.failJobsWhenReviewNeeded == 'true' }}
if: ${{ steps.codeball_status.outputs.approved == 'false' && inputs.failJobsWhenReviewNeeded == 'true' }}

@krlvi krlvi force-pushed the action-feature-flags branch from 0d59d17 to 0de630d Compare June 13, 2022 13:26
@krlvi krlvi requested a review from zegl June 13, 2022 13:27
@krlvi krlvi merged commit ee6d6bc into main Jun 13, 2022
@krlvi krlvi deleted the action-feature-flags branch June 13, 2022 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants