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

chore: relative links for contribution guidelines #19715

Merged
merged 1 commit into from Jan 22, 2023

Conversation

lmilbaum
Copy link
Contributor

@lmilbaum lmilbaum commented Jan 7, 2023

Changes

Context

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

@CLAassistant
Copy link

CLAassistant commented Jan 7, 2023

CLA assistant check
All committers have signed the CLA.

@lmilbaum lmilbaum force-pushed the contribution branch 2 times, most recently from d59324f to 324e752 Compare January 7, 2023 11:24
@rarkins
Copy link
Collaborator

rarkins commented Jan 7, 2023

Thanks for the PR. How confident are you that these relative links work in every case?

@lmilbaum
Copy link
Contributor Author

lmilbaum commented Jan 7, 2023

Thanks for the PR. How confident are you that these relative links work in every case?

I am confident with the md files not in the .github folder. Not confident with the others. Didn't figure out yet how to validate them. Any idea?

@rarkins
Copy link
Collaborator

rarkins commented Jan 7, 2023

We can maybe manually edit some existing ones and test if it still works correctly afterwards

@lmilbaum lmilbaum marked this pull request as ready for review January 7, 2023 13:39
@lmilbaum lmilbaum marked this pull request as draft January 7, 2023 13:41
@lmilbaum lmilbaum marked this pull request as ready for review January 7, 2023 13:42
@lmilbaum
Copy link
Contributor Author

Shell I close the PR?

.github/pull_request_template.md Outdated Show resolved Hide resolved
.github/label-actions.yml Outdated Show resolved Hide resolved
rarkins
rarkins previously approved these changes Jan 22, 2023
@rarkins rarkins enabled auto-merge (squash) January 22, 2023 09:12
auto-merge was automatically disabled January 22, 2023 17:58

Head branch was pushed to by a user without write access

@lmilbaum lmilbaum requested a review from rarkins January 22, 2023 17:59
@rarkins rarkins enabled auto-merge (squash) January 22, 2023 19:31
@rarkins rarkins merged commit 2b1ae24 into renovatebot:main Jan 22, 2023
@renovate-release
Copy link
Collaborator

🎉 This PR is included in version 34.109.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lmilbaum lmilbaum deleted the contribution branch January 23, 2023 08:38
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants