Skip to content

Update lib package dependencies using yarn #40

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

Merged
merged 3 commits into from
May 16, 2025
Merged

Conversation

didierkrux
Copy link
Member

npm install was not working with a fresh install but I managed to make it work using yarn instead.

Copy link

netlify bot commented May 15, 2025

Deploy Preview for devcon-monorepo ready!

Name Link
🔨 Latest commit 8a6fa20
🔍 Latest deploy log https://app.netlify.com/projects/devcon-monorepo/deploys/6826fa4788d51f0008b19124
😎 Deploy Preview https://deploy-preview-40--devcon-monorepo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented May 15, 2025

Deploy Preview for devcon-archive-monorepo ready!

Name Link
🔨 Latest commit 8a6fa20
🔍 Latest deploy log https://app.netlify.com/projects/devcon-archive-monorepo/deploys/6826fa48b974c50008003d40
😎 Deploy Preview https://deploy-preview-40--devcon-archive-monorepo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented May 15, 2025

Deploy Preview for devconnect-monorepo failed.

Name Link
🔨 Latest commit 8a6fa20
🔍 Latest deploy log https://app.netlify.com/projects/devconnect-monorepo/deploys/6826fa474418c30008a9c4b8

Copy link

netlify bot commented May 15, 2025

Deploy Preview for devcon-app ready!

Name Link
🔨 Latest commit 8a6fa20
🔍 Latest deploy log https://app.netlify.com/projects/devcon-app/deploys/6826fa48eb11dc0008e13ea6
😎 Deploy Preview https://deploy-preview-40--devcon-app.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented May 15, 2025

Deploy Preview for devcon-archive ready!

Name Link
🔨 Latest commit 8a6fa20
🔍 Latest deploy log https://app.netlify.com/projects/devcon-archive/deploys/6826fa48384ed10008b0b4fc
😎 Deploy Preview https://deploy-preview-40--devcon-archive.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented May 15, 2025

Deploy Preview for devcon-social ready!

Name Link
🔨 Latest commit 8a6fa20
🔍 Latest deploy log https://app.netlify.com/projects/devcon-social/deploys/6826fa4881b360000884fc12
😎 Deploy Preview https://deploy-preview-40--devcon-social.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@lassejaco
Copy link
Collaborator

Nice, can you try to remove the package-lock.json (and confirm it works after doing so with a clean install)? Ideally we only have one lockfile to avoid confusion in the future

PS. the CI errors in github can be ignored

Copy link

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert (click for details)
Warn Critical
[email protected] has a Critical CVE.

CVE: GHSA-f82v-jwr5-mffw Authorization Bypass in Next.js Middleware (CRITICAL)

Affected versions: >= 13.0.0, < 13.5.9

Patched version: 13.5.9

From: devcon-app/package.jsonnpm/[email protected]

ℹ Read more on: This package | This alert | What is a critical CVE?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at [email protected].

Suggestion: Remove or replace dependencies that include known critical CVEs. Consumers can use dependency overrides or npm audit fix --force to remove vulnerable dependencies.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/[email protected]. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@didierkrux
Copy link
Member Author

@lassejaco clean install seems to be working now after removing the package-lock.json file.
I also updated the README files.

@lassejaco lassejaco merged commit f6b3107 into main May 16, 2025
26 of 30 checks passed
@didierkrux didierkrux deleted the lib-install-fix branch May 16, 2025 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants