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: display warnings even if no errors #191

Merged
merged 1 commit into from
Sep 24, 2018

Conversation

GuillaumeAmat
Copy link
Contributor

Currently no warnings are displayed until there is at least one error. That PR fixes that.

Copy link
Collaborator

@denar90 denar90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💪

@denar90 denar90 merged commit 432b65c into paulirish:master Sep 24, 2018
@GuillaumeAmat GuillaumeAmat deleted the fix-warnings branch September 25, 2018 05:16
@GuillaumeAmat
Copy link
Contributor Author

@denar90 Do you know when that fix will be released?

@denar90
Copy link
Collaborator

denar90 commented Oct 4, 2018

Sorry, I didn't mention, it was released right after merging your PR.

@GuillaumeAmat
Copy link
Contributor Author

Hmmm... No it's not ^^

The last version is the 4.0.0, released 22 days ago, or that PR has been merged 10 days ago.

@denar90
Copy link
Collaborator

denar90 commented Oct 4, 2018

Strange that you don't see new version, because on npm it's 4.0.1 - https://www.npmjs.com/package/pwmetrics
also there is a proper version in package.json
https://github.com/paulirish/pwmetrics/blob/master/package.json#L3

@denar90
Copy link
Collaborator

denar90 commented Oct 4, 2018

I think the confusion here because I didn't create a tag...
Fixed https://github.com/paulirish/pwmetrics/releases/tag/v4.0.1 😄

@GuillaumeAmat
Copy link
Contributor Author

I think the confusion here because I didn't create a tag...

Damned... I should have looked at NPM too 😊
Thanks @denar90

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.

2 participants