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 action interpolation syntax #14

Merged
merged 2 commits into from Oct 10, 2019
Merged

Fix action interpolation syntax #14

merged 2 commits into from Oct 10, 2019

Conversation

tenhobi
Copy link
Contributor

@tenhobi tenhobi commented Oct 10, 2019

Fix action interpolation syntax #13

@tenhobi tenhobi self-assigned this Oct 10, 2019
@tenhobi tenhobi requested a review from ad-m October 10, 2019 09:54
@tenhobi tenhobi mentioned this pull request Oct 10, 2019
@ad-m ad-m requested a review from pdcmoreira October 10, 2019 19:20
@tenhobi tenhobi merged commit 9ab1401 into master Oct 10, 2019
@delete-merged-branch delete-merged-branch bot deleted the tenhobi-patch-1 branch October 10, 2019 19:41
@ad-m
Copy link
Owner

ad-m commented Oct 10, 2019

It breaks build:


1s
##[error]Fail to load /home/runner/work/_actions/ad-m/git-push-action/master/action.yml
Current runner version: '2.159.0'
Prepare workflow directory
Prepare all required actions
Download action repository 'actions/checkout@v1'
Download action repository 'ad-m/git-push-action@master'
##[error](Line: 9, Col: 18):
##[error](Line: 9, Col: 18): Unrecognized named-value: 'secrets'. Located at position 1 within expression: secrets.GITHUB_TOKEN
##[error]Fail to load /home/runner/work/_actions/ad-m/git-push-action/master/action.yml

@tenhobi
Copy link
Contributor Author

tenhobi commented Oct 10, 2019

What build does it breaks? The Docker Image CI actions, I dont see any issue on those? :O

@ad-m
Copy link
Owner

ad-m commented Oct 10, 2019

@tenhobi , private build using that action.

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.

None yet

2 participants