Skip to content

Conversation

@Skn0tt
Copy link
Contributor

@Skn0tt Skn0tt commented Jun 23, 2023

Description

In getBundleWeight, we were interpreting filenames as glob patterns. This failed for files like [show-id].js.
Since getBundleWeight was only ever passed full filenames, i'm removing the glob part in this PR.

Documentation

Tests

You can test this change yourself like so:

  1. TODO

Relevant links (GitHub issues, etc.) or a picture of cute animal

https://app.bugsnag.com/netlify/netlify-build/errors/64935ebec1a1aa00088f511c?event_id=64935ebe00bec5c72c8f0000&i=sk&m=nw

@netlify
Copy link

netlify bot commented Jun 23, 2023

Deploy Preview for netlify-plugin-nextjs-static-root-demo ready!

Name Link
🔨 Latest commit 9c75083
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-static-root-demo/deploys/64d152b8c33f8a000879ec24
😎 Deploy Preview https://deploy-preview-2181--netlify-plugin-nextjs-static-root-demo.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.

@netlify
Copy link

netlify bot commented Jun 23, 2023

Deploy Preview for netlify-plugin-nextjs-nx-monorepo-demo ready!

Name Link
🔨 Latest commit 9c75083
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-nx-monorepo-demo/deploys/64d152b8ea92e000083058de
😎 Deploy Preview https://deploy-preview-2181--netlify-plugin-nextjs-nx-monorepo-demo.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.

@netlify
Copy link

netlify bot commented Jun 23, 2023

Deploy Preview for netlify-plugin-nextjs-export-demo ready!

Name Link
🔨 Latest commit 9c75083
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-export-demo/deploys/64d152b8170a41000802d7c5
😎 Deploy Preview https://deploy-preview-2181--netlify-plugin-nextjs-export-demo.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.

@github-actions github-actions bot added the type: bug code to address defects in shipped code label Jun 23, 2023
@netlify
Copy link

netlify bot commented Jun 23, 2023

Deploy Preview for netlify-plugin-nextjs-demo ready!

Name Link
🔨 Latest commit 9c75083
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-demo/deploys/64d152b9cfa6bc00089a0068
😎 Deploy Preview https://deploy-preview-2181--netlify-plugin-nextjs-demo.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.

@netlify
Copy link

netlify bot commented Jun 23, 2023

Deploy Preview for nextjs-plugin-custom-routes-demo ready!

Name Link
🔨 Latest commit 9c75083
🔍 Latest deploy log https://app.netlify.com/sites/nextjs-plugin-custom-routes-demo/deploys/64d152b9618a8b00084f8f69
😎 Deploy Preview https://deploy-preview-2181--nextjs-plugin-custom-routes-demo.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.

@netlify
Copy link

netlify bot commented Jun 23, 2023

Deploy Preview for netlify-plugin-nextjs-next-auth-demo ready!

Name Link
🔨 Latest commit 9c75083
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-next-auth-demo/deploys/64d152b9a64ca400082cb6c7
😎 Deploy Preview https://deploy-preview-2181--netlify-plugin-nextjs-next-auth-demo.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.

@netlify
Copy link

netlify bot commented Jun 23, 2023

Deploy Preview for next-plugin-edge-middleware ready!

Name Link
🔨 Latest commit 9c75083
🔍 Latest deploy log https://app.netlify.com/sites/next-plugin-edge-middleware/deploys/64d152b9000214000809198c
😎 Deploy Preview https://deploy-preview-2181--next-plugin-edge-middleware.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.

@netlify
Copy link

netlify bot commented Jun 23, 2023

Deploy Preview for next-plugin-canary ready!

Name Link
🔨 Latest commit 9c75083
🔍 Latest deploy log https://app.netlify.com/sites/next-plugin-canary/deploys/64d152b955a40c0008583eb0
😎 Deploy Preview https://deploy-preview-2181--next-plugin-canary.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.

@netlify
Copy link

netlify bot commented Jun 23, 2023

Deploy Preview for next-i18next-demo ready!

Name Link
🔨 Latest commit 9c75083
🔍 Latest deploy log https://app.netlify.com/sites/next-i18next-demo/deploys/64d152b96dec7d0008a6e22b
😎 Deploy Preview https://deploy-preview-2181--next-i18next-demo.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.

@Skn0tt Skn0tt self-assigned this Jun 26, 2023
@nickytonline nickytonline self-assigned this Aug 7, 2023
@Skn0tt Skn0tt closed this Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug code to address defects in shipped code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants