Skip to content

CONTRIBUTING.rst has broken links when viewed on GitHub #1910

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

Closed
pushfoo opened this issue Sep 23, 2023 · 6 comments
Closed

CONTRIBUTING.rst has broken links when viewed on GitHub #1910

pushfoo opened this issue Sep 23, 2023 · 6 comments

Comments

@pushfoo
Copy link
Member

pushfoo commented Sep 23, 2023

Bug Report

tl;dr :ref:s and other directives don't seem to render in a GitHub view of an .rst file, even for targets within the same file

System Info

Version 117.0.5938.62 (Official Build) built on Debian 11.7, running on Debian 11.6 (64-bit)

Actual behavior:

The Testing link in After making changes does not link anywhere and does not render correctly:
image

Expected behavior:

The Testing link in After making changes:

  1. Works when clicked
  2. (Ideally) renders cleanly

Steps to reproduce/example code:

  1. Go to After making changes
  2. Observe the link
  3. Attempt to click it

Correcting the capitalization does not change this either. It appears GitHub cannot render :ref: targets.

Root causes

There seems to be limited post-processing of directives in GitHub views

Implications

This may block #1302 and affect #1886, both of which I was looking at when I noticed this.

Possible ways to fix it

  1. Use a script which renders CONTRIBUTING.rst as markdown
  2. Go back to old approach: the abbreviated model + links to the web site

I'm not in favor of clever scripts because those will probably break at some point.

@cspotcode
Copy link
Collaborator

Go back to old approach: the abbreviated model + links to the web site

It can be more than abbreviated, it can be a single link to the website with zero additional information.

@pushfoo
Copy link
Member Author

pushfoo commented Sep 23, 2023

I'm ok with that option, but might make it more or less a toctree dump for the relevant pages. @pvcraven any objections? I don't see the value of keeping this page seperate then reaching back through the hierarchy to include it.

@einarf
Copy link
Member

einarf commented Nov 18, 2023

Can't we just change README and CONTRIBUTING into markdown files? They don't really need the complexity rst has to offer in the first place.

@Cleptomania
Copy link
Member

Actually the CONTRIBUTING file is included directly in the doc build, though it would be trivial to add in MyST Parser extension to sphinx to allow us to just include the Markdown(and maybe use more markdown in the future?)

@einarf
Copy link
Member

einarf commented Jan 16, 2024

Why don't we just link to the file instead and make it markdown? That has such a lower complecity. It makes sese that the contribution guidelines are part of the code anyway and not the docs. This is pretty standard in other projects.

@pushfoo
Copy link
Member Author

pushfoo commented Mar 15, 2024

This seems done. We switched to markdown for the current file.

@pushfoo pushfoo closed this as completed Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants