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: fix isCssModulesEnabled check when styles is undefined and improve setup in README #1237

Merged
merged 1 commit into from
Oct 8, 2020

Conversation

skvale
Copy link
Contributor

@skvale skvale commented Oct 8, 2020

Description

fix isCssModulesEnabled check when styles is undefined and improve setup in README

fixes #issueid

@skvale skvale self-assigned this Oct 8, 2020
@netlify
Copy link

netlify bot commented Oct 8, 2020

Deploy preview for fundamental-react ready!

Built with commit 97a993e

https://deploy-preview-1237--fundamental-react.netlify.app

@skvale
Copy link
Contributor Author

skvale commented Oct 8, 2020

why are my builds hanging forever

@prsdthkr
Copy link
Member

prsdthkr commented Oct 8, 2020

Yep, this seems to be happening a lot recently. I've restarted the build.

@jbadan
Copy link
Contributor

jbadan commented Oct 8, 2020

@skvale @prsdthkr It's because all of SAP open source share five builds only on travis...

average wait time is 41 minutes this week 🙄

Screen Shot 2020-10-08 at 12 23 09 PM

@prsdthkr
Copy link
Member

prsdthkr commented Oct 8, 2020

Eeeeshhh 🙄

@prsdthkr
Copy link
Member

prsdthkr commented Oct 8, 2020

I think moving to travis.com will reduce build times.
image

@jbadan
Copy link
Contributor

jbadan commented Oct 8, 2020

@prsdthkr For a little bit, but once everyone moves over after .org is no longer supported it will be even worse.

@skvale skvale merged commit 7b90a32 into master Oct 8, 2020
@skvale skvale deleted the fix/fd-react-without-css-modules branch October 8, 2020 20:29
@prsdthkr
Copy link
Member

prsdthkr commented Oct 8, 2020

@droshev pointed this out

🤯
image

@jbadan
Copy link
Contributor

jbadan commented Oct 8, 2020

@prsdthkr I've opened multiple issues in their repo asking them to work on their build times and they don't want to haha

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.

4 participants