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

fix: Commit messages aren't fully shell escaped #165

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

amondnet
Copy link
Owner

close #164
close #162

Signed-off-by: Minsu Lee [email protected]

@amondnet amondnet linked an issue Jun 27, 2022 that may be closed by this pull request
@amondnet amondnet self-assigned this Jun 27, 2022
@github-actions
Copy link

github-actions bot commented Jun 27, 2022

Deploy preview for zeit-now-deployment-action-example-static ready!

✅ Preview
https://zeit-now-deployment-action-example-static-kspbkfuf8-amond.vercel.app
https://staging.static.vercel-action.amond.dev
https://pr-165.static.vercel-action.amond.dev

Built with commit d25df76.
This pull request is being automatically deployed with vercel-action

@github-actions
Copy link

github-actions bot commented Jun 27, 2022

Deploy preview for team-scope-test ready!

✅ Preview
https://team-scope-test-anwmr8tqp-dietfriends.vercel.app

Built with commit d25df76.
This pull request is being automatically deployed with vercel-action

@github-actions
Copy link

github-actions bot commented Jun 27, 2022

Deploy preview for express-basic-auth ready!

✅ Preview
https://express-basic-auth-pcrztjkcb-amond.vercel.app

Built with commit d25df76.
This pull request is being automatically deployed with vercel-action

@amondnet amondnet force-pushed the 164-commit-messages-arent-fully-shell-escaped branch from 9ef9eef to 9b20fa7 Compare June 27, 2022 02:01
@github-actions
Copy link

github-actions bot commented Jun 27, 2022

Deploy preview for vercel-action-example-nextjs ready!

✅ Preview
https://vercel-action-example-nextjs-kdhqt8oht-amond.vercel.app
https://staging.nextjs.vercel-action.amond.dev
https://pr-165.nextjs.vercel-action.amond.dev

Built with commit d25df76.
This pull request is being automatically deployed with vercel-action

@github-actions
Copy link

github-actions bot commented Jun 27, 2022

Deploy preview for zeit-now-deployment-action-example-angular ready!

✅ Preview
https://zeit-now-deployment-action-example-angular-qb155nb8k-amond.vercel.app
https://staging.angular.vercel-action.amond.dev
https://pr-165.angular.vercel-action.amond.dev

Built with commit d25df76.
This pull request is being automatically deployed with vercel-action

`foo`, 'bar', "baz"

close #164
close #162

Signed-off-by: Minsu Lee <[email protected]>
@amondnet amondnet force-pushed the 164-commit-messages-arent-fully-shell-escaped branch from 9b20fa7 to 8ed64b8 Compare June 27, 2022 02:03
`app` , "test", 'foo'

Signed-off-by: Minsu Lee <[email protected]>
`app` , "test", 'foo'

close #164
close #162

Signed-off-by: Minsu Lee <[email protected]>
@amondnet amondnet force-pushed the 164-commit-messages-arent-fully-shell-escaped branch from 13b5b55 to b818161 Compare June 27, 2022 02:20
`app` , "test", 'foo'

Signed-off-by: Minsu Lee <[email protected]>
`app` , "test", 'foo'

Signed-off-by: Minsu Lee <[email protected]>
@sonarcloud
Copy link

sonarcloud bot commented Jun 27, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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

Successfully merging this pull request may close these issues.

Commit messages aren't fully shell escaped Action failed for Bad function name
1 participant