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: always pass compilation to issues hook #443

Merged
merged 1 commit into from
Jun 10, 2020

Conversation

arthurdandrea
Copy link
Contributor

@arthurdandrea arthurdandrea commented Jun 10, 2020

Currently on a watch run of webpack this plugin is not passing the compilation argument to the issues hook. This PR fixes that.

Related: the type of the issues hook explicitly defines compilation a possible undefined, maybe if this is approved it could be changed as well.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 23721 lines exceeds the maximum allowed for the inline comments feature.

@codeclimate
Copy link

codeclimate bot commented Jun 10, 2020

Code Climate has analyzed commit 8a30aa7 and detected 56 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 27
Duplication 29

View more on Code Climate.

@arthurdandrea arthurdandrea changed the base branch from master to beta June 10, 2020 00:30
@piotr-oles
Copy link
Collaborator

Thank you for the PR, looks good :)

@piotr-oles piotr-oles merged commit b865f1d into TypeStrong:beta Jun 10, 2020
@piotr-oles
Copy link
Collaborator

🎉 This PR is included in version 5.0.0-beta.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

@piotr-oles
Copy link
Collaborator

🎉 This PR is included in version 5.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants