-
Notifications
You must be signed in to change notification settings - Fork 147
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
Update jobs to push ggshield formula on both GitGuardian's taps #145
Update jobs to push ggshield formula on both GitGuardian's taps #145
Conversation
a7a65a7
to
4bac2dd
Compare
Thanks for the review Aurélien, should be better now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now, thanks! 👍
.github/workflows/tag.yml
Outdated
- name: Checkout Homebrew-ggshield | ||
uses: actions/checkout@master | ||
with: | ||
repository: GitGuardian/homebrew-ggshield | ||
token: ${{ secrets.PAT_GITHUB }} | ||
path: ./brew | ||
path: ./ggshield |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am surprised, don't you want to checkout it in the brew directory like it was done before?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We checkout this repo to ./ggshield
now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is already a ggshield directory in the repo root, don't you want to use another non existent one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be good now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
4bac2dd
to
911c722
Compare
911c722
to
d8ee302
Compare
Context
In this MR, we want to lay the ground for migrating to a central tap for all GitGuardian's tool.
What we did
This MR simply modifies the
tag
workflow to push ggshield formula to two taps.Warning : We should wait for
GitGuardian/homebrew-tap
to be public before merging.