Skip to content

Commit

Permalink
Two monitor-components fixes (#4168)
Browse files Browse the repository at this point in the history
The `monitor-components` workflow's purpose is to notify the Git for
Windows maintainers when new package versions are published and may need
to be integrated into Git for Windows.

This PR brings two improvements to this workflow:

- We now explicitly disable the workflow everywhere but the Git for
Windows fork of Git, instead of relying on scheduled workflows being
disabled in forks (of the fork) by default.
- The label and the URL of the Git Credential Manager part was fixed. I
noticed that the label was wrong when
git-for-windows/build-extra#451 was opened but
failed to mention that it would close the issue
(#4166)
  • Loading branch information
dscho authored Dec 16, 2022
2 parents de9501c + 450817f commit 1396a3d
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions .github/workflows/monitor-components.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@ env:

jobs:
job:
# Only run this in Git for Windows' fork
if: github.event.repository.owner.login == 'git-for-windows'
runs-on: ubuntu-latest
permissions:
issues: write
Expand All @@ -26,8 +28,8 @@ jobs:
feed: https://github.com/git/git/tags.atom
- label: git-lfs
feed: https://github.com/git-lfs/git-lfs/tags.atom
- label: gcm-core
feed: https://github.com/microsoft/git-credential-manager-core/tags.atom
- label: git-credential-manager
feed: https://github.com/GitCredentialManager/git-credential-manager/tags.atom
- label: tig
feed: https://github.com/jonas/tig/tags.atom
- label: cygwin
Expand Down

0 comments on commit 1396a3d

Please sign in to comment.