-
Notifications
You must be signed in to change notification settings - Fork 42
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
Define GitKraken CLI in tools #577
Conversation
EliSchleifer
commented
Dec 7, 2023
•
edited
Loading
edited
- Define GitKraken CLI in tools
- Enable shell-hooks in the repo
⏱️ 53m total CI duration on this PR
|
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.
Mainly need to fix the shell_hooks bit
.trunk/trunk.yaml
Outdated
path_add: | ||
- "${workspace}/tools" |
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.
Please remove this bit, it's not relevant here
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
shell_hooks:
enforce: true
tools/gk/gk.test.ts
Outdated
@@ -0,0 +1,12 @@ | |||
import { makeToolTestConfig, toolTest } from "tests"; | |||
|
|||
toolTest({ |
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.
prefer toolInstallTest
. We're in the process of migrating things over. It streamlines this boilerplate
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.
LGTM
- Define GitKraken CLI in tools - Enable shell-hooks in the repo