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

remove module and rootDir from the compilerOptions #27

Merged
merged 1 commit into from
Nov 2, 2020

Conversation

nirjan-dev
Copy link
Contributor

  • Fixes typescript issue while using another svelte component from npm
  • Fixes an issue with prop exports in svelte files that use typescript
  • related issue

@nickreese
Copy link
Contributor

@NK-WebDev I know @thisislawatts is working on a typescript change as well. Will these conflict? Elderjs/elderjs#72

@nirjan-dev
Copy link
Contributor Author

@nickreese He hasn't changed this file in his commits so far but I don't know if he plans to change it latter or if these config values will need to be changed after merging his changes, so for now I can include a link to this PR on the PR for the docs (for ts info). Then We can change the docs later if the typescript setup process changes.

@nickreese nickreese merged commit db45d1c into Elderjs:master Nov 2, 2020
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