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

Add singapore to deploy matrix #23

Merged
merged 3 commits into from
Sep 14, 2023
Merged

Conversation

adolsalamanca
Copy link
Contributor

@adolsalamanca adolsalamanca commented Sep 14, 2023

  • Adds singapore to the deploy matrix
  • Stop downloading actionlin on each run, use a pre-built action instead.
  • Fixes lint in this actions.

@adolsalamanca adolsalamanca changed the title add singapore to matrix Add singapore to deploy matrix Sep 14, 2023
@adolsalamanca adolsalamanca force-pushed the adol/add-singapore-to-matrix branch 2 times, most recently from 304114e to c70bf5a Compare September 14, 2023 11:02
@adolsalamanca adolsalamanca requested a review from a team September 14, 2023 11:05
Comment on lines 21 to 25
- uses: reviewdog/action-actionlint@v1
with:
actionlint_flags: -ignore SC2086 -config-file .github/actionlint.yaml
fail_on_error: true
level: error
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

@adolsalamanca adolsalamanca merged commit 6037410 into main Sep 14, 2023
1 check passed
@adolsalamanca adolsalamanca deleted the adol/add-singapore-to-matrix branch September 14, 2023 11:18
adolsalamanca added a commit that referenced this pull request Sep 14, 2023
* add singapore to matrix

* Use actionlint instead of download it

* fix lint
adolsalamanca added a commit that referenced this pull request Sep 14, 2023
* add singapore to matrix

* Use actionlint instead of download it

* fix lint
adolsalamanca added a commit that referenced this pull request Sep 14, 2023
* add singapore to matrix

* Use actionlint instead of download it

* fix lint
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.

2 participants