Skip to content

fix: dirname not found in import.meta#866

Open
9aoy wants to merge 2 commits into
mainfrom
fix/import.meta
Open

fix: dirname not found in import.meta#866
9aoy wants to merge 2 commits into
mainfrom
fix/import.meta

Conversation

@9aoy

@9aoy 9aoy commented Jan 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

fix dirname not found in import.meta.

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings January 14, 2026 08:29
@netlify

netlify Bot commented Jan 14, 2026

Copy link
Copy Markdown

Deploy Preview for rstest-dev ready!

Name Link
🔨 Latest commit 6827b0f
🔍 Latest deploy log https://app.netlify.com/projects/rstest-dev/deploys/69675403b71fcb00084d1881
😎 Deploy Preview https://deploy-preview-866--rstest-dev.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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +212 to +213
meta.dirname = path.dirname(fileName);
meta.filename = fileName;

Copilot AI Jan 14, 2026

Copy link

Choose a reason for hiding this comment

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

The ImportMeta interface in importMeta.d.ts should be updated to include type definitions for the newly added dirname and filename properties. This will provide proper TypeScript support for these properties.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants