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(build): Support building Gatsby app under Windows #1248

Merged
merged 3 commits into from
Jan 29, 2019
Merged

fix(build): Support building Gatsby app under Windows #1248

merged 3 commits into from
Jan 29, 2019

Conversation

TimoStaudinger
Copy link

What:
Fixes #1246.

Windows uses backslashes instead of the Unix style forward slashes to separate file path segments.
This caused problems with the Gatsby build of the PF4 react-docs package.

These changes should resolve this by taking both types of slashes into account in the build scripts.

@TimoStaudinger TimoStaudinger changed the title fix(build): Support building Gatsby app under Windows WIP: fix(build): Support building Gatsby app under Windows Jan 25, 2019
@patternfly-build
Copy link
Contributor

PatternFly-React preview: https://1248-pr-patternfly-react-patternfly.surge.sh

@coveralls
Copy link

coveralls commented Jan 25, 2019

Pull Request Test Coverage Report for Build 4125

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 80.368%

Totals Coverage Status
Change from base Build 4107: 0.0%
Covered Lines: 4552
Relevant Lines: 5329

💛 - Coveralls

@TimoStaudinger TimoStaudinger changed the title WIP: fix(build): Support building Gatsby app under Windows fix(build): Support building Gatsby app under Windows Jan 25, 2019
Copy link
Collaborator

@jschuler jschuler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @timosta

Copy link
Contributor

@tlabaj tlabaj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dlabaj dlabaj merged commit 89bc7f8 into patternfly:master Jan 29, 2019
@TimoStaudinger TimoStaudinger deleted the fix/windows-gatsby-build branch January 29, 2019 18:38
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.

Running yarn develop in react-docs fails under Windows
7 participants