Skip to content

Conversation

@santoshyadavdev
Copy link
Contributor

@santoshyadavdev santoshyadavdev commented Apr 21, 2020

A similar PR may already be submitted!
Please search among the Pull request before creating one.

Thanks for submitting a pull request! Please provide enough information so that others can review your pull request.

Summary

This PR fixes/implements the following bugs/features

  • update to Angular v9
  • Breaking changes
@santoshyadavdev santoshyadavdev marked this pull request as draft April 21, 2020 12:08
@orizens
Copy link
Owner

orizens commented Apr 29, 2020

@santoshyadav198613 thanks for raising this pr. is that ready for review?

@santoshyadavdev santoshyadavdev marked this pull request as ready for review May 2, 2020 02:33
@santoshyadavdev
Copy link
Contributor Author

Hi @orizens ,
This is ready for review now.

@orizens
Copy link
Owner

orizens commented May 3, 2020

@santoshyadav198613 have you verified unit tests are passing?

@santoshyadavdev
Copy link
Contributor Author

image
yes working can you add github action so it can be verified on each pull request

@orizens
Copy link
Owner

orizens commented May 4, 2020

travis build was failing. i fixed it. should be good now.

@orizens
Copy link
Owner

orizens commented May 4, 2020

@santoshyadav198613 can you advice: should there be any breaking changes?

@santoshyadavdev
Copy link
Contributor Author

santoshyadavdev commented May 4, 2020

@santoshyadav198613 can you advice: should there be any breaking changes?

No breaking changes the update should be smooth.

@santoshyadavdev
Copy link
Contributor Author

Hi @orizens ,
Can we merge this?

@orizens orizens merged commit ead8568 into orizens:master May 13, 2020
@orizens
Copy link
Owner

orizens commented May 13, 2020

thanks @santoshyadav198613
v9.0.0 is live now.

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

Labels

None yet

2 participants