-
Notifications
You must be signed in to change notification settings - Fork 188
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
Modernize plugin #847
Modernize plugin #847
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
9d7d37b
to
3a8aadf
Compare
I scaled back the modernization a tiny bit so I could get the tests passing. We can modernize more as we go along. |
3a8aadf
to
8c1bfb1
Compare
8c1bfb1
to
1d64503
Compare
I'm not sure why this would fail on linux only. Do you have any context? |
@rahulsom,
|
@rahulsom, |
Thanks for the nudge. I'm closing this. |
When I started, the master branch did not build.
Error
This change gets the build working.
passed.