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

Fixes #28243 - upgrade vendor to v3 #474

Merged
merged 2 commits into from
Dec 19, 2019
Merged

Conversation

sharvit
Copy link
Contributor

@sharvit sharvit commented Nov 12, 2019

The main upgrades are redux-hooks. babel v7 and jest v24

Copy link
Contributor

@adamruzicka adamruzicka left a comment

Choose a reason for hiding this comment

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

Works nicely, but travis is unhappy

@sharvit
Copy link
Contributor Author

sharvit commented Nov 20, 2019

Rebased, it should fix the linting issue

@sharvit
Copy link
Contributor Author

sharvit commented Nov 24, 2019

Updated to use @theforeman/babel

Copy link

@jeremylenz jeremylenz left a comment

Choose a reason for hiding this comment

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

Tested and LGTM, thanks @sharvit :)

@jeremylenz
Copy link

Why is Jenkins not happy? Shows 0 failures

@adamruzicka
Copy link
Contributor

The tests are failing because of rubocop. #480 needs to get in first and this PR needs to be rebased on top of that

@sharvit
Copy link
Contributor Author

sharvit commented Dec 3, 2019

Thanks, rebased!

Copy link
Contributor

@adamruzicka adamruzicka left a comment

Choose a reason for hiding this comment

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

Works nicely, ACK

@ares
Copy link
Member

ares commented Dec 19, 2019

Thanks @sharvit and @adamruzicka, merging this and will release 1.0.0 for nightlies.

@ares ares merged commit 74ee387 into theforeman:master Dec 19, 2019
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.

5 participants