Skip to content

Add main property to package.json #3

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

Closed
wants to merge 2 commits into from

Conversation

ctaepper
Copy link

@ctaepper ctaepper commented Aug 7, 2016

having the main property follows npm best practices for simpler including the main file of the package.

for example, when using cssloader in webpack

/* before */
.container {
  composes: db from 'tachyons-display/src/tachyons-display.css';
}

/* after */
.container {
  composes: db from 'tachyons-display';
}

i'll try to do PRs for all modules, any help is appreciated... otherwise, please try to let me know rather sooner that later, if that work would be wasted, because you don't want that feature :)

@ctaepper
Copy link
Author

ctaepper commented Aug 7, 2016

PS: i'm not gonna add any PR descriptions to the other modules, i assume once is enough

@ctaepper
Copy link
Author

ctaepper commented Aug 7, 2016

forgive me for being blind, there is already the style property for this path. however i'm not sure i can webpack make evaluate this porperty? maybe @sokra could shed some light?

@ctaepper
Copy link
Author

ctaepper commented Aug 7, 2016

apparently its talked about here: webpack-contrib/css-loader#51

@ctaepper ctaepper closed this Aug 7, 2016
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.

1 participant