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: import error on Windows (fix #1) #3

Merged
merged 1 commit into from
Feb 1, 2022
Merged

Conversation

hugoattal
Copy link
Collaborator

Fix #1

Description

Windows has a hard time with slash and backslash on file paths. I used absolute path and pathe to make sure it uses slash.
Histoire now work flawlessly on Windows 🥳 !

What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

@hugoattal hugoattal added the bug Something isn't working label Feb 1, 2022
@hugoattal hugoattal requested a review from Akryum February 1, 2022 21:13
@hugoattal hugoattal self-assigned this Feb 1, 2022
@netlify
Copy link

netlify bot commented Feb 1, 2022

❌ Deploy Preview for histoirejs failed.

🔨 Explore the source changes: 59741a5

🔍 Inspect the deploy log: https://app.netlify.com/sites/histoirejs/deploys/61f9a2773d5cf800078ba079

@hugoattal hugoattal changed the title Fix import error on Windows (fix #1) fix: import error on Windows (fix #1) Feb 1, 2022
@hugoattal
Copy link
Collaborator Author

@Akryum I don't have access to netlify logs, I'm not quite sure why the deployment failed...
Also, what would you like for such PR? I wait for your review or simply merge it and move on?

@Akryum
Copy link
Member

Akryum commented Feb 1, 2022

Don't look at the netlify build, the doc site is not ready yet

@Akryum Akryum merged commit 2ed7d42 into main Feb 1, 2022
@Akryum Akryum deleted the fix-1-window-import-error branch February 15, 2022 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Internal server error when launching on Windows
2 participants