-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fix cicd #1
Fix cicd #1
Conversation
.github/workflows/test.yml
Outdated
node-version: '18' | ||
|
||
- name: Install dependencies | ||
run: npm install --frozen-lockfile |
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 saw that you have a bun.lock
file on the root of the repo, which makes me think that you're using bun instead of npm. Is that right?
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.
True 🙆♂️
on: workflow_dispatch | ||
on: | ||
push: | ||
branches: [main] |
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.
do you ever push to main?
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.
Not ideally. But can happen hueuheuehu
@@ -30,5 +43,5 @@ jobs: | |||
|
|||
- name: Run Forge tests | |||
run: | | |||
forge test -vvv | |||
forge test --fork-url ${{secrets.RPC_OP}} -vvv --via-ir |
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.
didn't know about the test's fork
flag, does this mean the tests are being run on the testnet?
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.
It means that, for the test scope, we will fork the network's state. We are not running it in the testnet, but locally.
Pull Request to fix the CI / CD powered by github actions (it was not broken).