Skip to content

Add GitHub Actions to check if a PR has a label and a milestone#41

Open
h-chmeruk wants to merge 3 commits intomainfrom
add-github-actions-pull_request_label
Open

Add GitHub Actions to check if a PR has a label and a milestone#41
h-chmeruk wants to merge 3 commits intomainfrom
add-github-actions-pull_request_label

Conversation

@h-chmeruk
Copy link
Contributor

Which issue(s) are closed by this pull request?

Closes #27

Changes proposed in this pull request:

  • Added files .github/workflows/has_label.html and .github/workflows/has_version_milestone.html. They were copied unchanged from the pyfar repository.
  • Extended the test function test_generated_file_exists() to include parameterization with new files.

@h-chmeruk h-chmeruk added this to the v0.1.0 milestone Feb 27, 2026
@h-chmeruk h-chmeruk added the enhancement New feature or request label Feb 27, 2026
@h-chmeruk h-chmeruk moved this from Backlog to Require review in Weekly Planning Feb 27, 2026
@h-chmeruk h-chmeruk requested a review from ahms5 February 27, 2026 08:21
Copy link
Member

@ahms5 ahms5 left a comment

Choose a reason for hiding this comment

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

nice thanks, just fixed a bug in the current implmentation

Comment on lines +7 to +8
- labeled
- unlabeled
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- labeled
- unlabeled
- milestoned
- demilestoned

ok thanks a bugfix from the current version. this will retrigger the workflow if we change the milestone instead of the label. See https://docs.github.com/de/actions/reference/workflows-and-actions/events-that-trigger-workflows#pull_request

Copy link
Member

Choose a reason for hiding this comment

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

we still need to change it here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, I just realized that I made this change in the file template-pyfar-package/.github/workflows/has_version_milestone.yml and not in the file template-pyfar-package/template/.github/workflows/has_version_milestone.yml. I’ll remove this change from the wrong file.

@h-chmeruk h-chmeruk requested a review from ahms5 March 6, 2026 08:57
Copy link
Member

@ahms5 ahms5 left a comment

Choose a reason for hiding this comment

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

almost done, it is just missing at an other position, toherwise approved

Comment on lines +7 to +8
- labeled
- unlabeled
Copy link
Member

Choose a reason for hiding this comment

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

we still need to change it here.

@h-chmeruk h-chmeruk requested a review from ahms5 March 20, 2026 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Require review

Development

Successfully merging this pull request may close these issues.

add other githubactions

2 participants