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 in PR template #5751

Merged
merged 1 commit into from
Mar 21, 2024
Merged

Docs: Update CLA link and add it in PR template #5751

merged 1 commit into from
Mar 21, 2024

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.

Copy link
Member

@vidorteg vidorteg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again! LGTM

@vidorteg
Copy link
Member

/azp run

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

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@davidlj95
Copy link
Contributor Author

/azp run

praying

😛

@vidorteg
Copy link
Member

Unfortunately it is still failing CI. I think that because it is modifying an .md file the linter step (to be more specific markdown-link-validator) is failing. I need to take a closer look to the markdown-link-validator usage to fix this and I'll ping you again 😅

@davidlj95
Copy link
Contributor Author

Unfortunately it is still failing CI. I think that because it is modifying an .md file the linter step (to be more specific markdown-link-validator) is failing. I need to take a closer look to the markdown-link-validator usage to fix this and I'll ping you again 😅

Yup, no problem at all! I'll take a look too if have some time to help a bit moar :)

@vidorteg
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vidorteg vidorteg merged commit 84b7d02 into webhintio:main Mar 21, 2024
5 checks passed
@vidorteg
Copy link
Member

well I was able to make it pass validation :). Thank you for the contribution.

@davidlj95
Copy link
Contributor Author

well I was able to make it pass validation :). Thank you for the contribution.

Awesome! Thanks to you and your perseverance ☺️

@davidlj95 davidlj95 deleted the update-cla-link branch March 21, 2024 20:25
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.

2 participants