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

Ignore directories with missing Chart.yaml #18

Merged
merged 1 commit into from
Apr 6, 2020
Merged

Ignore directories with missing Chart.yaml #18

merged 1 commit into from
Apr 6, 2020

Conversation

ccremer
Copy link
Contributor

@ccremer ccremer commented Mar 24, 2020

Prevents action fails when there are directories detected that do not contain charts (e.g. .github)

as proposed in #12

feedback welcome

fixes #12

@ccremer
Copy link
Contributor Author

ccremer commented Mar 24, 2020

An example test run here: https://github.com/ccremer/charts/runs/531250478?check_suite_focus=true

    - name: Run chart-releaser
      uses: ccremer/chart-releaser-action@ignore-dirs
      env:
        CR_TOKEN: "${{ secrets.CR_TOKEN }}"
      with:
        charts_dir: .

Log:

Looking up latest tag...
Discovering changed charts since 'znapzend-0.3.1'...
Warning: .github/Chart.yaml is missing, assuming that '.github' is not a helm chart. Skipping.
Warning: README.md/Chart.yaml is missing, assuming that 'README.md' is not a helm chart. Skipping.
Installing chart-releaser...
Packaging chart 'znapzend'...
Successfully packaged chart and saved it to: .cr-release-packages/znapzend-0.3.1.tgz
Releasing charts...

Copy link
Member

@unguiculus unguiculus left a comment

Choose a reason for hiding this comment

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

This makes sense, thanks. I made a few comments.

cr.sh Outdated Show resolved Hide resolved
cr.sh Outdated Show resolved Hide resolved
cr.sh Outdated Show resolved Hide resolved
@ccremer
Copy link
Contributor Author

ccremer commented Apr 4, 2020

Thanks for the review! I updated my commit.

cr.sh Outdated Show resolved Hide resolved
cr.sh Show resolved Hide resolved
Prevents action fails when there are directories detected that do not contain charts.
(e.g. .github)

Signed-off-by: Chris <github.account@chrigel.net>
@ccremer
Copy link
Contributor Author

ccremer commented Apr 5, 2020

Added your suggestions.
Example output (commit contains a change in README.md, which is not logged now):

Looking up latest tag...
Discovering changed charts since 'znapzend-0.3.1'...
WARNING: .github/Chart.yaml is missing, assuming that '.github' is not a Helm chart. Skipping.
Installing chart-releaser...
Packaging chart 'znapzend'...
Successfully packaged chart and saved it to: .cr-release-packages/znapzend-0.3.1.tgz
Releasing charts...

@scottrigby
Copy link
Member

I added a fixes comment to the description above to automatically close related issue on PR merge

@unguiculus unguiculus merged commit 8505b7a into helm:master Apr 6, 2020
@SamuAlfageme
Copy link

Would be nice to have a v1.0.0-rc.3 with whis one in. For repos with:

        with:
          charts_dir: .

... this fails Packaging chart '.github' (the dir containing the action's workflow itself)

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

Successfully merging this pull request may close these issues.

Ignore hidden folders when detecting charts or only package Charts with chart.yaml
5 participants