-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
#3840 - allow user to pass any path separator as --chunk_output_path_prefix #3988
#3840 - allow user to pass any path separator as --chunk_output_path_prefix #3988
Conversation
…output_path_prefix
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Looks like the CLA bot still doesn't recognize you. Did you follow the steps in https://github.com/google/closure-compiler/pull/3988/checks?check_run_id=7995613406 |
hi, as i am part of the POC within dynatrace regarding Open Source contribution, I thought it is worth to add what we did so far, maybe we (Point of contacts) screwed up or missed a step :): Corporate signers
We're not sure what we might have missed and we highly appreciate guidance regarding this, so we can move forward 🙂 |
Hi. Sorry for long wait. We've never had to troubleshoot a CLA issue in a while, so weren't sure how. I just followed our internal docs.
@aepfli , can I ask you to check one more time if:
If everything looks correct on your side, then I will need to file an issue internally. Maybe the CLA bot tripped up on something. |
hi, sorry for the long wait on my side I checked, and @DerGernTod and his email gernot.raudner@dynatrace.com is part of our google-contributors group as you can see on the screenshot. should we check something else @frigus02? i am not sure if we might miss something here |
Thanks for confirming, @aepfli. I filed an issue internally with the CLA team. I'm going to keep you updated here. |
@aepfli, can you share the full name (i.e. email address) of the "google-contributors" group shown in the screenshot? It is possible that the group of authorized contributors should be something like |
@aepfli, yes that's what I was talking about. A colleague informed me that this is not the Google Group associated with your CLA. The currently associated group is |
Thank you @frigus02 for the help, I am now investigating who might have access to this google group, and I also try to set it to the workspace group :) THANK YOU, very much! |
No worries. Feel free to ping me here if you're stuck. |
dee780a
to
09fe64d
Compare
09fe64d
to
55e3c55
Compare
CLA should work now, i changed the commit message in the previous commit to retrigger the CLA bot |
Good to hear you got the CLA working. Re-assigned to Bradford, as he already reviewed previous attempt. He's also more familiar with the code than I am. |
This looks good. |
i recreated this PR since somehow the CLA bot didn't recognize me
fixes #3840 - allow user to pass any path separator as --chunk_output_path_prefix
as a user, you might want to use unix-style path separators always, since they're also compatible with windows. doing so however causes issues when writing source maps, since maybeCreateDirsForPath only creates output directories if path separators are in os-style.