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

fix(cli): remove filename of input #1602

Closed
wants to merge 1 commit into from
Closed

Conversation

lzt1008
Copy link
Contributor

@lzt1008 lzt1008 commented Sep 15, 2022

When using [...fileCache]to convert Map to an array, the key of fileCache as filename, will also be passed. That's not what we want.

@netlify
Copy link

netlify bot commented Sep 15, 2022

Deploy Preview for unocss ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit b2ddb5f
🔍 Latest deploy log https://app.netlify.com/sites/unocss/deploys/6323115e011670000873864f
😎 Deploy Preview https://deploy-preview-1602--unocss.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 settings.

@zyyv
Copy link
Member

zyyv commented Sep 16, 2022

@lzt1008 #1592 also handle a part of the this code, you can reveiew it. I think it would be better to merge the commits together

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.

None yet

2 participants