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

Use correct 'main' file in package.json #83

Closed
wants to merge 1 commit into from

Conversation

Josiah
Copy link

@Josiah Josiah commented May 25, 2014

The package.json file doesn't contain the correct 'main' field which means that you can't just require('nprogress'). This PR will resolve this.

The `package.json` file doesn't contain the correct 'main' field which means that you can't just `require('nprogress')`. This PR will resolve this.
@caarlos0
Copy link

+1

1 similar comment
@liquidvisual
Copy link

+1

@frankie-loves-jesus
Copy link

Does anyone know Rico personally and can confirm that he's alright? He hasn't been seen in over a month.

@callumacrae
Copy link

He's still active on GitHub, look at his profile.

I merged this PR to my fork and pushed to npm as "nprogress-npm": https://www.npmjs.org/package/nprogress-npm

@frankie-loves-jesus
Copy link

Yeah. @joscha could you reopen #72?

@joscha
Copy link

joscha commented Jun 20, 2014

@frankie-loves-jesus I can't, it was closed by the owner @rstacruz...

@callumacrae
Copy link

Pretty sure you can reopen it unless it has been locked

@joscha
Copy link

joscha commented Jun 20, 2014

@callumacrae I think it has, the button is not there...

@callumacrae
Copy link

You can't talk in locked conversations. Weird, they must have changed it so that reporters can't reopen them.

@rstacruz
Copy link
Owner

This has been fixed in 0.1.5. Thanks folks! You should now be able to use nprogress with browserify and webpack.

@rstacruz rstacruz closed this Jun 25, 2014
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.

7 participants