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

chore(ci): fix bash condition to publish chart versions #17321

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Conversation

atomrc
Copy link
Contributor

@atomrc atomrc commented Apr 25, 2024

Description

The condition to generate a prod/staging release was wrong. Bash needs [[ in order to use regexp. see https://unix.stackexchange.com/questions/306111/what-is-the-difference-between-the-bash-operators-vs-vs-vs

Copy link

sonarcloud bot commented Apr 25, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.15%. Comparing base (aeda4ba) to head (587ad2d).

Additional details and impacted files
@@           Coverage Diff           @@
##              dev   #17321   +/-   ##
=======================================
  Coverage   46.14%   46.15%           
=======================================
  Files         751      751           
  Lines       24732    24732           
  Branches     5659     5659           
=======================================
+ Hits        11413    11414    +1     
  Misses      11886    11886           
+ Partials     1433     1432    -1     

@atomrc atomrc merged commit 85f161a into dev Apr 25, 2024
11 checks passed
@atomrc atomrc deleted the fix/bash branch April 25, 2024 10:21
V-Gira added a commit that referenced this pull request May 22, 2024
)

Co-authored-by: Thomas Belin <thomasbelin4@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants