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 script (as mentioned in https://github.com/StephenChou1017/react-big-scheduler/issues/56#issuecomment-455448991) #91

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bl5ck
Copy link

@bl5ck bl5ck commented Jan 18, 2019

Learn more here (#56 (comment))
Please feel free to let me know if you need any adjustment/information.

@StephenChou1017
Copy link
Owner

@bl5ck
Hi, thanks for the PR. Is it used to resolve this issue(or else?):
After I merge your code, I can execute the npm run build without errors when adding ~ in src/less/antd-globals-hiding-hack.less file?

@bl5ck
Copy link
Author

bl5ck commented Jan 19, 2019

Actually you can but without adding ~. The error in my case is because I don't install execution binary globally. But that won't work with npm run example. It may need some adjustments though.

@bl5ck
Copy link
Author

bl5ck commented Jan 19, 2019

Let me know if you need that too. Then I would find another way to combine those fixes.

@StephenChou1017
Copy link
Owner

@bl5ck
What execution binary that you don't install globally causes the error in your case? I can't reappear your error using my latest repo at present, and I need to reappear it first.

@bl5ck
Copy link
Author

bl5ck commented Jan 22, 2019

Mines were babel and lessc.

@bl5ck
Copy link
Author

bl5ck commented Jan 22, 2019

Also please note that I use ubuntu 18.04

@hbatalhaStch
Copy link

hbatalhaStch commented Mar 13, 2023

hi @mathildamonni are you planning on resuming work on this repo

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