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: windows unit test fix #3027

Merged
merged 3 commits into from
Oct 8, 2021
Merged

fix: windows unit test fix #3027

merged 3 commits into from
Oct 8, 2021

Conversation

Vandivier
Copy link
Contributor

  1. change host to a OS-agnostic host (localhost)
  2. update lock file
  3. normalize deck2 content to remove windows failure due to line endings

closes #3025

fix for windows
line ending difference caused failure
now simple.md content mirror deck1
hakimel added a commit that referenced this pull request Oct 8, 2021
@hakimel hakimel merged commit 6b15e6c into hakimel:master Oct 8, 2021
@hakimel
Copy link
Owner

hakimel commented Oct 8, 2021

Thanks for the PR—I wasn't aware 0.0.0.0 doesn't working in Windows. There shouldn't be any issues switching to localhost but if anyone notices anything let me know.

The changes you made to the markdown test caused them to break in macOS so I've rolled that back for the time being.

shogunsea pushed a commit to shogunsea/reveal.js that referenced this pull request Oct 24, 2021
shogunsea added a commit to shogunsea/reveal.js that referenced this pull request Oct 24, 2021
* 'master' of github.com:shogunsea/reveal.js:
  revert changes to md test hakimel#3027
  fix: deck 2 horizontal separator
  chore: lock update
  fix: host fix for windows
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.

0.0.0.0 fails on windows
2 participants