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

Internal changes to the charm shouldn't be released #89

Merged
merged 2 commits into from
Jun 27, 2023
Merged

Conversation

dragomirp
Copy link
Contributor

Issue

Charm is published to charmhub on each push to main even if there are no changes to the charm itself

Solution

Try to limit the amount of releases based on the content of the push

@@ -6,6 +6,12 @@ on:
push:
branches:
- main
paths-ignore:
- 'tests/**'
- 'docs/**'
Copy link
Contributor Author

@dragomirp dragomirp Jun 21, 2023

Choose a reason for hiding this comment

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

There are no docs here yet.

Copy link
Contributor

Choose a reason for hiding this comment

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

On the way! :-D

@dragomirp dragomirp marked this pull request as ready for review June 26, 2023 17:49
@@ -6,6 +6,12 @@ on:
push:
branches:
- main
paths-ignore:
- 'tests/**'
- 'docs/**'
Copy link
Contributor

Choose a reason for hiding this comment

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

On the way! :-D

@dragomirp dragomirp merged commit 2fd4d4c into main Jun 27, 2023
14 checks passed
@dragomirp dragomirp deleted the limit-publish branch June 27, 2023 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants