Skip to content
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(deps): update all to latest version #15

Merged
merged 1 commit into from
Nov 28, 2023
Merged

Conversation

maninak
Copy link
Collaborator

@maninak maninak commented Nov 28, 2023

Unfortunately still no fix for this

nuxt-modules/icon#46 (comment)

that I hoped for and we need to re-enable typechecking on CI.

Signed-off-by: Konstantinos Maninakis <maninak@protonmail.com>
Copy link

netlify bot commented Nov 28, 2023

Deploy Preview for chipper-wisp-c7553e ready!

Name Link
🔨 Latest commit 40e213f
🔍 Latest deploy log https://app.netlify.com/sites/chipper-wisp-c7553e/deploys/6565f6c1c268bb00088edcbf
😎 Deploy Preview https://deploy-preview-15--chipper-wisp-c7553e.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 site configuration.

@maninak maninak self-assigned this Nov 28, 2023
@maninak maninak added the chore Tooling or infra-related, not directly affecting core product label Nov 28, 2023
Copy link
Collaborator

@QZera QZera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After merging with main locally, I get this error on pnpm dev.
Everything seems to run just fine though.
Is this something to be concerned about?

 ERROR  [worker reload] Cannot read properties of null (reading 'address')                                                                                                                                                                      7:31:34 PM

  at _reload (node_modules/.pnpm/nitropack@2.8.0/node_modules/nitropack/dist/shared/nitro.37241ef0.mjs:282:38)
  at runNextTicks (node:internal/process/task_queues:60:5)
  at process.processTimers (node:internal/timers:511:9)
  at async _applyPromised (node_modules/.pnpm/perfect-debounce@1.0.0/node_modules/perfect-debounce/dist/index.mjs:54:10)

@maninak
Copy link
Collaborator Author

maninak commented Nov 28, 2023

After merging with main locally, I get this error on pnpm dev. Everything seems to run just fine though. Is this something to be concerned about?

 ERROR  [worker reload] Cannot read properties of null (reading 'address')                                                                                                                                                                      7:31:34 PM

  at _reload (node_modules/.pnpm/nitropack@2.8.0/node_modules/nitropack/dist/shared/nitro.37241ef0.mjs:282:38)
  at runNextTicks (node:internal/process/task_queues:60:5)
  at process.processTimers (node:internal/timers:511:9)
  at async _applyPromised (node_modules/.pnpm/perfect-debounce@1.0.0/node_modules/perfect-debounce/dist/index.mjs:54:10)

Good catch. According to the maintainer it seems to be safe to ignore and should be resolved soon.

Copy link
Collaborator

@QZera QZera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@maninak maninak merged commit 0fb53e6 into main Nov 28, 2023
8 checks passed
@maninak maninak deleted the chore/update-deps branch November 28, 2023 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Tooling or infra-related, not directly affecting core product
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants