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

tests(generators): plugins util tests #1448

Merged
merged 4 commits into from
Apr 12, 2020

Conversation

knagaitsev
Copy link
Contributor

@knagaitsev knagaitsev commented Apr 12, 2020

What kind of change does this PR introduce?

tests

Did you add tests for your changes?

Yes

If relevant, did you update the documentation?

No

Summary

generators plugins util tests

Does this PR introduce a breaking change?

No

Other information

@knagaitsev knagaitsev requested a review from a team as a code owner April 12, 2020 00:28
Copy link
Member

@anshumanv anshumanv left a comment

Choose a reason for hiding this comment

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

Some tests seem to fail, can you check?

@knagaitsev
Copy link
Contributor Author

Some tests seem to fail, can you check?

It seems like a bad test run, nothing in my changes is causing the problem as far as I can tell

@knagaitsev knagaitsev closed this Apr 12, 2020
@knagaitsev knagaitsev reopened this Apr 12, 2020
@anshumanv
Copy link
Member

It seems like a bad test run, nothing in my changes is causing the problem as far as I can tell

Yes just saw, unrelated serve test is failing. Re-running should fix.

@knagaitsev
Copy link
Contributor Author

Let me push another commit, I forgot to test one case anyway

@anshumanv
Copy link
Member

Unusual, init-generator tests seem to be timing out now.

@webpack-bot
Copy link

@evilebottnawi Thanks for your update.

I labeled the Pull Request so reviewers will review it again.

@anshumanv Please review the new changes.

@alexander-akait alexander-akait merged commit 44298f3 into webpack:next Apr 12, 2020
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

6 participants