Skip to content

Commit

Permalink
Merge pull request #588 from rstudio/do-not-test-install-rstudio-library
Browse files Browse the repository at this point in the history
Exclude rstudio-library, you cannot install library charts
  • Loading branch information
jforest authored Oct 16, 2024
2 parents bcf3579 + 42cc879 commit 269e9c1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/chart-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -129,14 +129,14 @@ jobs:
- name: Run chart-testing (install changed)
id: ct-install
if: ${{ github.ref != 'refs/heads/main' && steps.list-changed.outputs.changed == 'true' }}
run: ct install --target-branch main --chart-dirs charts --chart-dirs other-charts
run: ct install --target-branch main --chart-dirs charts --chart-dirs other-charts --excluded-charts charts/rstudio-library
continue-on-error: true

# no allow-failure until https://github.com/actions/toolkit/issues/399
- name: Run chart-testing (install all)
id: ct-install-all
if: ${{ github.ref == 'refs/heads/main' }}
run: ct install --target-branch main --all --chart-dirs charts --chart-dirs other-charts
run: ct install --target-branch main --all --chart-dirs charts --chart-dirs other-charts --excluded-charts charts/rstudio-library
continue-on-error: true

- name: Notify Slack of chart install failure
Expand Down

0 comments on commit 269e9c1

Please sign in to comment.