Update LogicBuilder.Expressions.Utils#187
Conversation
|
@BlaiseD OK to merge? 😄 |
|
@matthewtquinn1 - we should be all set once you sign the CLA. |
|
@BlaiseD Thank you. had a look at the CLA - it says the author must sign it; would that be the author of the repo? Or this PR? |
Must be the PR. The CLA check only come up on your first contribution. |
|
@BlaiseD Sorry I've never seen this before. Is there a guide for how to sign it? |
|
@microsoft-github-policy-service agree |
|
This might help. Here's another resource. |
|
@BlaiseD That first link no longer works, they say that it is an old and unused method of doing it now ;( The second link, I see it suggests how to accept it here but I followed that step and it still hasn't passed [https://cla2.dotnetfoundation.org/](https://github.com/microsoft/contributorlicenseagreement#accepting) |
|
Here's an example from another PR. |
|
@matthewtquinn1 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information. @dotnet-policy-service agree [company="Full Staq Ltd"] (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer. @dotnet-policy-service agree company="Full Staq Ltd" |
|
@dotnet-policy-service agree company="Full Staq Ltd" |
|
@BlaiseD Got it! Ready to merge :) |
|
@jbogard - is it possible part of the CLA workflow is missing for AutoMapper.Extensions.OData? I suspect the prompt/instructions were not clear/visible for @matthewtquinn1 compared with e.g. this PR from the Expression Mapping repo. |
Resolves #186