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.

Sign up for GitHub

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

Jump to bottom

Update jsx-ast-utils to fix #327 #356

Closed
wants to merge 2 commits into from
Closed

Update jsx-ast-utils to fix #327 #356

wants to merge 2 commits into from

Conversation

Waize
Copy link

@Waize Waize commented Nov 14, 2017

@ljharb
Copy link
Member

ljharb commented Nov 14, 2017

This isn't actually needed, since ^2.0.0 already includes 2.0.1.

@Waize
Copy link
Author

Waize commented Nov 14, 2017

true but this "bug" is really annoying
is there any plan, to rebuild the package with the new version?

@ljharb
Copy link
Member

ljharb commented Nov 14, 2017

That's up to the owner.

@coveralls
Copy link

coveralls commented Nov 14, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 32cc614 on Waize:master into 356944a on evcohen:master.

@Waize
Copy link
Author

Waize commented Nov 14, 2017

so we will wait for @evcohen

@jnwng jnwng mentioned this pull request Dec 7, 2017
Closed
@Waize Waize closed this Dec 14, 2017
@Waize
Copy link
Author

Waize commented Dec 14, 2017

Close due to new version bump

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Waize @ljharb @coveralls