Skip to content

Conversation

pdurbin
Copy link
Member

@pdurbin pdurbin commented Sep 18, 2025

How to preview this change

Go to https://dataverse-guide--11831.org.readthedocs.build/en/11831/contributor/documentation.html

What this PR does / why we need it:

My laptop got wiped and I learned that Python 3.10 is required these days (due to Sphinx-Substitution-Extensions being added in #11477). I made sure this is mentioned.

Also, I've talked on and off with @DieuwertjeBloemen about running Sphinx locally. I rewrote the "installing locally" section and suggested trying Docker first.

Finally, I added a tip about how to make a custom title when linking from a .md file.

Which issue(s) this PR closes:

None.

Special notes for your reviewer:

I don't think we can add the Python 3.10 requirement to requirements.txt but I'm open to it.

As to how to make a pull request, these docs are still punting to the main developer docs. I guess that's ok but I'm open to maintaining a simplified version here.

Also, should the contributor clone the main repo or their fork? And if they clone their fork, how do they pull the latest from upstream? I like how Zulip is very prescriptive on what they want you to do. See https://zulip.readthedocs.io/en/latest/git/cloning.html#get-zulip-code and https://zulip.readthedocs.io/en/latest/git/using.html#keep-your-fork-up-to-date

Related:

@pdurbin pdurbin added the Size: 0.5 A percentage of a sprint. 0.35 hours label Sep 18, 2025
@pdurbin pdurbin moved this to Ready for Review ⏩ in IQSS Dataverse Project Sep 18, 2025
@cmbz cmbz added FY26 Sprint 6 FY26 Sprint 6 (2025-09-10 - 2025-09-24) FY26 Sprint 7 FY26 Sprint 7 (2025-09-24 - 2025-10-08) labels Sep 24, 2025
@coveralls
Copy link

coveralls commented Sep 26, 2025

Coverage Status

coverage: 23.538%. remained the same
when pulling 4a95207 on doc-contrib
into 56bf628 on develop.

@cmbz cmbz added the FY26 Sprint 8 FY26 Sprint 8 (2025-10-08 - 2025-10-22) label Oct 8, 2025
@stevenwinship stevenwinship self-assigned this Oct 10, 2025
@stevenwinship stevenwinship moved this from Ready for Review ⏩ to In Review 🔎 in IQSS Dataverse Project Oct 10, 2025
`docker run -it --rm -v $(pwd):/docs sphinxdoc/sphinx:7.2.6 bash -c "cd doc/sphinx-guides && pip3 install -r requirements.txt && make html"`

### Installing GraphViz
If all goes well, you should be able open `doc/sphinx-guides/build/html/index.html` to see the guides you just built.
Copy link
Contributor

Choose a reason for hiding this comment

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

change to: be able to open

stevenwinship

This comment was marked as duplicate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FY26 Sprint 6 FY26 Sprint 6 (2025-09-10 - 2025-09-24) FY26 Sprint 7 FY26 Sprint 7 (2025-09-24 - 2025-10-08) FY26 Sprint 8 FY26 Sprint 8 (2025-10-08 - 2025-10-22) Size: 0.5 A percentage of a sprint. 0.35 hours

Projects

Status: In Review 🔎

Development

Successfully merging this pull request may close these issues.

4 participants