-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Remove stdout: prefix #2373
Remove stdout: prefix #2373
Conversation
Fixes #2095, by disabling the `stdout:` prefix with the browser shim.
see dicsussion in #2095. if there's a better alternative than |
@skeggse I realize this was sent over a year ago, but was wondering if you could sign the CLA? |
Happy to sign the CLA. Unfortunately, it appears to be incomplete. It references a document called |
cc @kborchers |
@skeggse FWIW I found the doc if you want to get it moving. 😉 |
@boneskull thank you. I've signed the CLA. |
@skeggse @boneskull Thanks! We are aware of the broken link and will fix ASAP. The main blocker is that currently, CLA Assistant treats any change to the CLA as a change that invalidates every previous signature. In general, that would be correct behavior to make sure anyone that has signed is made aware of a change and has to sign again to agree to the change. In this case, it's the equivalent of a typo but the application doesn't know the difference so we are trying to formulate a procedure to make the change and migrate all of the signatures over to the "new" version with minimal (preferably no) impact to the projects. |
@skeggse thanks! |
Fixes #2095, by disabling the
stdout:
prefix with the browser shim.