Skip to content

fix: Make the link to CONTRIBUTING.md absolute to fix dead linking in PR descriptions - #94

Merged
antalszava merged 1 commit into
mainfrom
fix_contributing_link_pr_template
Aug 18, 2026
Merged

fix: Make the link to CONTRIBUTING.md absolute to fix dead linking in PR descriptions#94
antalszava merged 1 commit into
mainfrom
fix_contributing_link_pr_template

Conversation

@antalszava

@antalszava antalszava commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

The link in this PR template at the end of the section is dead linking to the Contributing page (when accessed from a PR's description):

Test plan


Important

Most code in ionq_core/ is auto-generated and overwritten on regeneration.
See CONTRIBUTING.md for which files are safe to edit.

@antalszava
antalszava requested review from natestemen and splch August 17, 2026 13:04
@antalszava
antalszava requested a review from a team as a code owner August 17, 2026 13:05

@natestemen natestemen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure I see the issue. If I go to https://github.com/ionq/ionq-core-python/blob/main/.github/pull_request_template.md in the browser, or open that file locally in vscode, the link works correctly.

@antalszava

Copy link
Copy Markdown
Contributor Author

@natestemen it doesn't work when clicking on the link from the PR description itself (its "main" location)

@antalszava
antalszava requested a review from natestemen August 18, 2026 07:26
@antalszava

Copy link
Copy Markdown
Contributor Author
image

@hodgestar-ionq hodgestar-ionq left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

+1 on the change.

Given that the template might appear in many kinds of places (e.g. a PR page as @antalszava mentioned) a relative link seems fragile.

@antalszava
antalszava force-pushed the fix_contributing_link_pr_template branch from 3a82d98 to a781856 Compare August 18, 2026 11:25
@antalszava antalszava changed the title fix: fix link to contributing markdown file in PR template fix: Make the link to CONTRIBUTING.md absolute to fix dead linking in PR descriptions Aug 18, 2026
@antalszava
antalszava merged commit dd89ff5 into main Aug 18, 2026
11 checks passed
@antalszava
antalszava deleted the fix_contributing_link_pr_template branch August 18, 2026 11:28
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