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: allow svelte.config.cjs #37

Merged
merged 2 commits into from
Apr 12, 2021
Merged

fix: allow svelte.config.cjs #37

merged 2 commits into from
Apr 12, 2021

Conversation

forrestjacobs
Copy link
Contributor

Addresses #35:

  • Update Cosmiconfig to v7, which allows "svelte.config.cjs"
  • Search for "svelte.config.cjs" as well as "svelte.config.js" when preprocess is true.

I don't love the test I added since it depends on the code that svelte generates, but I didn't want to make it too complex. Let me know if you want me to make changes. I'm not very available for the next 48 hours so if you want to make changes yourself or if someone else opens a better PR, I'm not too precious about my contribution -- I just selfishly want to be able to use svelte.config.cjs myself!

@mihar-22 mihar-22 merged commit 9e259c4 into svelteness:master Apr 12, 2021
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.

2 participants