fix(@ngtools/webpack): one star replacement should be preceded by slash#4455
fix(@ngtools/webpack): one star replacement should be preceded by slash#4455hccampos wants to merge 1 commit intoangular:masterfrom hccampos:fix-exclude
Conversation
|
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). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
|
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. |
|
I signed it! |
|
@hccampos any reason your email address does not match the CLA email? this commit was done under hugo.campos@pix4d.com which no CLA has been signed for. |
|
Yes, I committed from my work account accidentally. I then changed it and
amended the commit but apparently GoogleBot didn't pick it up.
…On Feb 7, 2017 01:51, "Hans" ***@***.***> wrote:
@hccampos <https://github.com/hccampos> any reason your email address
does not match the CLA email? this commit was done under
***@***.*** which no CLA has been signed for.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4455 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ADlcCmScynJRyvBrogyNgWniWHwOo4c5ks5rZ8ASgaJpZM4L4MCT>
.
|
|
@hccampos can you resubmit this PR please? It seems googlebot isn't getting it. |
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
When building a project with AoT enabled, I noticed that
.spec.tsfiles were not being excluded properly. This was happening because the pattern**/*.spec.tstriggered the two-stars replacement:And then the one star replacement would replace the
*inserted by the previous replacement:With this PR, the single-star replacement expects a slash before the star, i.e.
/*.spec.ts, so:Which then properly excludes the spec files. This doesn't handle the case where the pattern is in the form of
**/something-*.spec.ts, but I am not sure that is super common.Since I am not super familiar with the codebase, If you guys know of a simpler or better way to fix this problem, I'd welcome feedback. Or feel free to implement/refactor yourselves.