Skip to content

[ReadTheDocs] Updating the documentation URLs from stable to last released#2173

Merged
vnktshr21 merged 2 commits intoni:masterfrom
rahur-NI:UpdateLatestVersionAsDefaultForRTD
Apr 7, 2026
Merged

[ReadTheDocs] Updating the documentation URLs from stable to last released#2173
vnktshr21 merged 2 commits intoni:masterfrom
rahur-NI:UpdateLatestVersionAsDefaultForRTD

Conversation

@rahur-NI
Copy link
Copy Markdown
Contributor

@rahur-NI rahur-NI commented Apr 2, 2026

  • This contribution adheres to CONTRIBUTING.md.
  • I've updated CHANGELOG.md if applicable.
  • I've added tests applicable for this pull request

What does this Pull Request accomplish?

This PR is to update all the ReadTheDocs documentation URLs in documentation template to not point to /stable
version for consistency across all drivers.
Before this fix, nidcpower and nirfsg had only /latest version (these were setup for independent versioning where ReadTheDocs redirects to last released version). But all other modules had /stable and /latest in which stable versioning will become outdated with the next release. This was causing inconsistency across the driver's documentation and the nirfsg link was bad.

After this fix, all the modules will be pointing to the last released version as default.

List issues fixed by this Pull Request below, if any.

None.

What testing has been done?

A successful tox execution with expected file change in README.rst

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.86%. Comparing base (f77dd57) to head (b9ca957).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2173      +/-   ##
==========================================
- Coverage   91.18%   89.86%   -1.32%     
==========================================
  Files          67       73       +6     
  Lines       16489    19004    +2515     
==========================================
+ Hits        15035    17078    +2043     
- Misses       1454     1926     +472     
Flag Coverage Δ
codegenunittests 84.90% <ø> (ø)
nidcpowersystemtests 94.65% <ø> (+0.04%) ⬆️
nidcpowerunittests 89.53% <ø> (ø)
nidigitalsystemtests 92.26% <ø> (ø)
nidigitalunittests 68.44% <ø> (ø)
nidmmsystemtests 92.72% <ø> (ø)
nifakeunittests 86.01% <ø> (ø)
nifgensystemtests 94.61% <ø> (ø)
nimodinstsystemtests 73.85% <ø> (ø)
nimodinstunittests 94.20% <ø> (ø)
nirfsgsystemtests 81.19% <ø> (?)
niscopesystemtests 92.94% <ø> (ø)
niscopeunittests 43.20% <ø> (ø)
nisesystemtests 91.50% <ø> (ø)
niswitchsystemtests 82.03% <ø> (ø)
nitclksystemtests 94.87% <ø> (ø)
nitclkunittests 98.26% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 7 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f77dd57...b9ca957. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rahur-NI rahur-NI marked this pull request as ready for review April 2, 2026 10:16
@ni-jfitzger
Copy link
Copy Markdown
Collaborator

Fixed some issues with the description. The PR will change what we're pointing to, but it won't delete or hide the stable version. Until our next release of APIs besides nidcpower and nirfsg, those other APIs will still redirect to stable. I'll have to make some changes in ReadTheDocs when I release them to hide the stable version and maybe to redirect (not sure). Right now those other APIs will redirect to stable, but that's okay because stable is still the last released version for them.

@ni-jfitzger ni-jfitzger changed the title [ReadTheDocs] Updating the documentation URLs from stable to latest [ReadTheDocs] Updating the documentation URLs from stable to last released Apr 2, 2026
@vnktshr21 vnktshr21 merged commit f17c461 into ni:master Apr 7, 2026
39 checks passed
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.

4 participants