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: #135 #140

Merged
Merged

Conversation

AnatoleLucet
Copy link
Collaborator

According to this issue: nodejs/node#31710, Node doesn't support backslash in an import. So I've just dropped the backslash when writing imports, but paths are still logged with backslash on windows (as they should).

@AnatoleLucet AnatoleLucet requested a review from benawad May 21, 2020 18:56
@codecov
Copy link

codecov bot commented May 21, 2020

Codecov Report

Merging #140 into develop will decrease coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #140      +/-   ##
===========================================
- Coverage    93.77%   93.73%   -0.04%     
===========================================
  Files           20       20              
  Lines          546      543       -3     
  Branches       123      121       -2     
===========================================
- Hits           512      509       -3     
  Misses          34       34              
Impacted Files Coverage Δ
src/index/generateTrees/buildGraph.ts 100.00% <ø> (ø)
src/index/formatFileStructure/fixImports.ts 97.22% <100.00%> (-0.15%) ⬇️
...x/formatFileStructure/fixImports/makeImportPath.ts 94.73% <100.00%> (-0.51%) ⬇️
src/index/generateTrees.ts 100.00% <100.00%> (ø)
src/index/getFilePaths.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7621940...8a2a1c0. Read the comment docs.

@AnatoleLucet AnatoleLucet merged commit 93b5ef8 into benawad:develop May 22, 2020
@andria-dev
Copy link

Is there any word on having this released as a patch version?

@AnatoleLucet AnatoleLucet deleted the fix/forward-slashes-on-windows branch May 23, 2020 19:56
@AnatoleLucet
Copy link
Collaborator Author

@ChrisBrownie55 I'll release it if you want.

@andria-dev
Copy link

@AnatoleLucet I'd really appreciate it! Can't currently use this project on Windows without out it lol

@github-actions
Copy link

🎉 This PR is included in version 0.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error: ENOENT: no such file or directory, scandir
3 participants