-
Notifications
You must be signed in to change notification settings - Fork 69
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
Updates remark-toc, remark-gfm, and rehype-autolink-headings libraries #1244
Conversation
@SahilKalra98 is attempting to deploy a commit to a Personal Account owned by @garageScript on Vercel. @garageScript first needs to authorize it. |
Codecov Report
@@ Coverage Diff @@
## master #1244 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 146 146
Lines 2454 2451 -3
Branches 638 638
=========================================
- Hits 2454 2451 -3
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/c0d3/c0d3-app/3LVAdMKMvQi165MJ6tHpuh4XA1gU |
package.json
Outdated
@@ -6,7 +6,7 @@ | |||
"scripts": { | |||
"db:init": "prisma migrate reset", | |||
"db:seed": "prisma db seed --preview-feature", | |||
"db:up": "docker-compose --file docker/docker-compose.yml up --detach postgres", | |||
"db:up": "sudo docker-compose --file docker/docker-compose.yml up --detach postgres", |
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.
its only sudo for your machine I think, on macOS and windows it should not be sudo. So I don't think the change is good
162d573
to
89b84f1
Compare
89b84f1
to
b7fa485
Compare
Closing PR because Next.js does not support esm |
NextJS has been updated to v12, it can be merged. |
@SahilKalra98 I reopened this. |
b7fa485
to
ddfe3cc
Compare
This PR updates the following three PRs:
1.#1208
2.#1236
3.#1246
The reason it updates all three is because all three were failing the same tests. Since all the packages were major version updates, they went from CommonJS modules to fully ESM modules. Our jest configuration doesn't support ESM modules yet so I mocked the three modules as a workaround.