Skip to content

Commit

Permalink
fix(generate.yml): replace double curly braces with single curly brac…
Browse files Browse the repository at this point in the history
…es in docker build and run commands to properly interpolate inputs.liquibase-version variable

chore(gitignore): add commands.json to .gitignore to exclude generated file from version control
  • Loading branch information
jandroav committed Mar 4, 2024
1 parent fedfb3f commit fc1dc0e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/generate.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,10 @@ jobs:
- uses: actions/checkout@v3

- name: Build liquibase-protobuf-generator
run: docker build -t liquibase-protobuf-generator:{{ inputs.liquibase-version }} . --build-arg VERSION={{ inputs.liquibase-version }}
run: docker build -t liquibase-protobuf-generator:${{ inputs.liquibase-version }} . --build-arg VERSION=${{ inputs.liquibase-version }}

- name: Generate liquibase commands
run: docker run --rm -v $(PWD):/proto liquibase-protobuf-generator:$(VERSION) --output-file=commands.json list-commands
run: docker run --rm -v $(PWD):/proto liquibase-protobuf-generator:${{ inputs.liquibase-version }} --output-file=commands.json list-commands

- uses: actions/upload-artifact@v3
with:
Expand Down
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ node_modules
.nyc_output
coverage
.npmrc
commands.json

# Mac OS
.DS_Store
Expand Down

0 comments on commit fc1dc0e

Please sign in to comment.