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: better check for unsafe config to fallback #839

Merged
merged 5 commits into from Apr 28, 2024

Conversation

ineshbose
Copy link
Collaborator

@ineshbose ineshbose commented Apr 24, 2024

  • Fix types (enhancements later) and provide more checking to moduleOptions.config
  • Check unsafe props from hooks and go to fallback
  • See if loading config can be moved out of context.ts

Planned for 6.12.1. Will close #838

Copy link

netlify bot commented Apr 24, 2024

Deploy Preview for nuxt-tailwindcss ready!

Name Link
🔨 Latest commit 118b17f
🔍 Latest deploy log https://app.netlify.com/sites/nuxt-tailwindcss/deploys/662d1e98400af40008967dcb
😎 Deploy Preview https://deploy-preview-839--nuxt-tailwindcss.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.

Copy link

what-the-diff bot commented Apr 24, 2024

PR Summary

  • Update to playground/nuxt.config.ts file
    Removed certain unnecessary resources hence increasing efficiency.
  • Overhaul of src/context.ts file
    Significant changes made for better tracking of changes and enhancing overall security.
  • Modified src/module.ts file
    Introduction of new features and improving older ones to enhance overall functionality. Added necessary warnings for better user experience.
  • Changes in src/types.ts file
    Enhancements made to increase the range of modifiable properties, lending more customizability to the users.

@ineshbose ineshbose changed the title chore: check for more options in config fix: better check for unsafe config to fallback Apr 24, 2024
@ineshbose ineshbose marked this pull request as ready for review April 27, 2024 15:52
@ineshbose ineshbose merged commit 9d78a9f into main Apr 28, 2024
6 of 7 checks passed
@ineshbose ineshbose deleted the chore/safer-config-checking branch April 28, 2024 09:06
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.

Safelist causing build failures
1 participant