-
Notifications
You must be signed in to change notification settings - Fork 66
chore: just scripts #1254
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
chore: just scripts #1254
Conversation
c8d0ee1
to
28d0994
Compare
08a9c99
to
6518523
Compare
CI error is because we need to install ni in CI:
|
I see. In that case, will |
We don't need That's a reasonable ask, just like we require everyone to install The alternative is to implement a Side note — maybe we should extend the vscode-setup repo and provide something like a general shellscript that @sablier-labs/everybody should run? |
Understand. In that case, we should add it under |
Bumping this up, in case you have missed my review @PaulRBerg, ignore otherwise. |
Sorry @smol-ninja haven't missed your review, just got bogged down with my Indexers refactor. I'll pick this up this week. Thanks for your review |
The PR is ready for re-review — and perhaps merge this time! @sablier-labs/evm |
build: bump deps fix: fix shub's pr feedback
Merging now — make sure you have |
Implements https://github.com/sablier-labs/command-center/issues/151
lmk if you like this. If yes, we can implement the same in the other repos
See https://github.com/sablier-labs/devkit