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.

Sign up for GitHub

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

Jump to bottom

ci/cd: notify action has a problem #716

Closed
kamilsk opened this issue Apr 17, 2023 · 0 comments
Closed

ci/cd: notify action has a problem #716

kamilsk opened this issue Apr 17, 2023 · 0 comments
Assignees
@kamilsk
Labels
effort: easy An issue is easy to implement. impact: medium An issue has medium impact. scope: inventory An issue related to auxiliary code, e.g. CI config, Makefiles, etc. severity: minor A bug has minor severity and should be fixed when possible. type: bug A new bug report.

Comments

@kamilsk
Copy link
Member

kamilsk commented Apr 17, 2023

Details

https://github.com/tact-app/web/actions/runs/4719810324/jobs/8371132735#step:4:20

if [ "true" == 'true' ]
  then
    echo txt="Revert "feat(deps): bump react-datepicker from 4.10.0 to 4.10.1 (#715)"
  
  This reverts commit 50b56fa0a3a9274d5de3f1b4d2ddefa8a5d45d77." | head -1 >> $GITHUB_OUTPUT
  elif [ "false" == 'true' ]
  then
    echo txt="regular healthcheck" >> $GITHUB_OUTPUT
  elif [ "false" == 'true' ]
  then
    echo txt="" >> $GITHUB_OUTPUT
  fi
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
/home/runner/work/_temp/ad41c761-f483-46eb-b22d-b36703bb890e.sh: line 3: syntax error near unexpected token `('

It should unescape quotes.
@kamilsk kamilsk added type: bug A new bug report. severity: minor A bug has minor severity and should be fixed when possible. scope: inventory An issue related to auxiliary code, e.g. CI config, Makefiles, etc. impact: medium An issue has medium impact. effort: easy An issue is easy to implement. labels Apr 17, 2023
@kamilsk kamilsk self-assigned this Apr 17, 2023
@github-project-automation github-project-automation bot moved this from Backlog to Done in Proof of Concept Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kamilsk kamilsk

Labels
effort: easy An issue is easy to implement. impact: medium An issue has medium impact. scope: inventory An issue related to auxiliary code, e.g. CI config, Makefiles, etc. severity: minor A bug has minor severity and should be fixed when possible. type: bug A new bug report.
Projects
Proof of Concept
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kamilsk