Skip to content
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

Docs: Update CLA link and add it to PR template #5691

Closed
wants to merge 1 commit into from
Closed

Docs: Update CLA link and add it to PR template #5691

wants to merge 1 commit into from

Conversation

davidlj95
Copy link
Contributor

Pull request checklist

Make sure you:

For non-trivial changes, please make sure you also:

  • N/A Added/Updated related documentation.
  • N/A Added/Updated related tests.

Short description of the change(s)

After looking for the CLA in docs for my first contribution found out the only reference to it had a link that didn't work (domain doesn't exist). Fixing that link + adding a link in PR template so impatient contributors like me can browse the CLA whilst the CLA signing comment appears.

@davidlj95 davidlj95 changed the title Update CLA link and add it to PR template Docs: Update CLA link and add it to PR template Aug 10, 2023
@molant
Copy link
Member

molant commented Aug 29, 2023

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vidorteg
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vidorteg
Copy link
Member

[Doing some cleaning]. Unfortunately and due to me missing to review this 😓the proposed url has gone also stalled. feel free to reopen pointing to the new one, or I can also do it if needed. thanks again for the contribution and sorry that this went missed.

@davidlj95
Copy link
Contributor Author

No problem at all @vidorteg :) just reopened with new link here: #5751

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants