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

add npm run build:css #1436

Merged
merged 4 commits into from May 8, 2016
Merged

add npm run build:css #1436

merged 4 commits into from May 8, 2016

Conversation

minrk
Copy link
Member

@minrk minrk commented May 7, 2016

setup.py css is just an alias for this, so there should be no more calls back to Python from npm run build:anything

closes #1433

@minrk minrk mentioned this pull request May 7, 2016
@rgbkrk
Copy link
Member

rgbkrk commented May 7, 2016

What an improvement. It looks like the bower fetch still needs to be run prior to building CSS.

@minrk minrk force-pushed the less-py-js-py-js-less branch 3 times, most recently from ba36231 to db0b5be Compare May 8, 2016 15:16
minrk added 4 commits May 8, 2016 16:33
setup.py css is just an alias for this,
so there should be no more calls back to Python from npm run build:anything
pip install runs these steps
@minrk
Copy link
Member Author

minrk commented May 8, 2016

After a bit of fiddling, this works now. less-2.7 fixes compatibility with node-6, but introduces a sourcemap bug (less/less.js#2896). We should upgrade LESS and restore source maps once they ship 2.7.1 with the fix.

@rgbkrk rgbkrk merged commit 4ec76aa into jupyter:master May 8, 2016
@ellisonbg
Copy link
Contributor

Great!

On Sun, May 8, 2016 at 9:03 AM, Kyle Kelley notifications@github.com
wrote:

Merged #1436 #1436.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#1436 (comment)

Brian E. Granger
Associate Professor of Physics and Data Science
Cal Poly State University, San Luis Obispo
@ellisonbg on Twitter and GitHub
bgranger@calpoly.edu and ellisonbg@gmail.com

@damianavila
Copy link
Member

@minrk 2.7.1 is available by now...

@minrk minrk added this to the 5.0 milestone May 30, 2016
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
5.0
Merged PRs
Development

Successfully merging this pull request may close these issues.

refactor npm run build:css
4 participants