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.

Sign up for GitHub

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

Jump to bottom

Expose scss file in "style" field of package.json #1045

Closed
ringods opened this issue Jun 11, 2016 · 1 comment
Closed

Expose scss file in "style" field of package.json #1045

ringods opened this issue Jun 11, 2016 · 1 comment
Labels
feature
Milestone
v3.3.7

Comments

@ringods
Copy link

ringods commented Jun 11, 2016

Hello,

I am trying use bootstrap-sass via npm, node-sass and the sass-module-importer. The importer documents this lookup strategy:

1. Set a SCSS/Sass/CSS file on the "main" field of their package.json/bower.json
2. Set a SCSS/Sass/CSS file on the "style" field of their package.json/bower.json
3. Have a index.css file on the root of their module

If I look at the bootstrap npm package, it uses the style field in package.json. It seems this field is becoming a de facto standard.

Is it possible to get a similar setup and expose the main scss file in the style field of package.json?
@cvrebert cvrebert changed the title Expose scss file in style field of package.json Expose scss file in "style" field of package.json Jun 17, 2016
@cvrebert
Copy link
Collaborator

We should also add a "sass" field for consistency with https://github.com/twbs/bootstrap/blob/ac1ea8c26ccb4b65d14bf207974854ca993ea4e5/package.json#L25

@marcofugaro marcofugaro mentioned this issue Jun 23, 2016
Merged
cvrebert added a commit that referenced this issue Jun 24, 2016
@cvrebert
Add "sass" field to package.json for consistency with upstream
6424fb7 
Refs #1045
pads pushed a commit to talis/bootstrap-sass that referenced this issue Jul 20, 2016
@danielmatthew @pads
Create distribution (#16)
64d294f 
* Add "style" field to package.json (twbs#1051); fixes twbs#1045

* Add "sass" field to package.json for consistency with upstream

Refs twbs#1045

* Create distribution
@glebm glebm added this to the v3.3.7 milestone Jul 25, 2016
@vic0707 vic0707 mentioned this issue Mar 14, 2017
Closed
ElMassimo pushed a commit to SolarCS/bootstrap-sass that referenced this issue Apr 11, 2017
@marcofugaro @ElMassimo
Add "style" field to package.json (twbs#1051); fixes twbs#1045
41eed0a
ElMassimo pushed a commit to SolarCS/bootstrap-sass that referenced this issue Apr 11, 2017
@cvrebert @ElMassimo
Add "sass" field to package.json for consistency with upstream
a9141d1 
Refs twbs#1045
@dependabot dependabot bot mentioned this issue Mar 6, 2021
Merged
This was referenced Mar 15, 2021
Open
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
v3.3.7
Development

No branches or pull requests
3 participants
@ringods @glebm @cvrebert