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

Fix version warning in documentation #2187

Merged
merged 5 commits into from
Oct 14, 2021
Merged

Conversation

steffengraber
Copy link
Contributor

Problem:
The documentation contains a warning prologue that should only appear in the "latest" version.

Solution:
Readthedocs contains an environment variable "READTHEDOCS_VERSION" at build time which contains the version number that is currently being built. Only if this is "latest" the warning is displayed.

Note: for locally built documentation no warning is shown anymore.

@heplesser heplesser added I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Critical Needs to be addressed immediately T: Bug Wrong statements in the code or documentation labels Oct 14, 2021
@heplesser heplesser added this to In progress in Documentation via automation Oct 14, 2021
@heplesser
Copy link
Contributor

A single reviewer should be enough for this one.

Documentation automation moved this from In progress to Review Oct 14, 2021
Copy link
Contributor

@jessica-mitchell jessica-mitchell left a comment

Choose a reason for hiding this comment

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

I think this will be helpful going forward with releases. We will keep in mind updating the version for each new release for local builds. thanks!

@heplesser heplesser merged commit e6c7bf0 into nest:master Oct 14, 2021
Documentation automation moved this from Review to Done Oct 14, 2021
@steffengraber steffengraber deleted the fix_prolog branch October 27, 2021 06:08
@clinssen
Copy link
Contributor

@steffengraber: I'm still seeing the warning on https://nest-simulator.readthedocs.io/en/v3.1/. Perhaps the v3.1 docs build needs to be manually triggered to incorporate the changes from this PR?

@steffengraber
Copy link
Contributor Author

@clinssen Unfortunately, we can not change the documentation for the current release. These changes will only take effect starting with the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Critical Needs to be addressed immediately T: Bug Wrong statements in the code or documentation
Projects
Documentation
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants