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

Display versioned manuals on nixos.org #1022

Closed
wants to merge 3 commits into from

Conversation

jlesquembre
Copy link
Member

Generate documentation for the latest 3 nix versions. 3 is a random choice, we can change how many versions we want to display.

Screenshot:

screenshot__2023-03-23_161527

Copy link
Member

@garbas garbas left a comment

Choose a reason for hiding this comment

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

👍 on the intent to show 5 version of documentation for Nix. And thank you for putting in the work @jlesquembre !

But we should improve the implementation, because it puts more work on the marketing team to keep everything in sync. I hope to explain this bellow.

In current situation Nix team has full control what gets deployed to the website, by just pushing to certain branches -> master and latest-release. Then an hourly github action will update the website.

I would like to keep this workflow.

As far as I can understand this PR, this would not be the case anymore, since if a patch release is done for older versions we would need to manually intervene and bump it. This also means that each release we would have to manually bump the versions (since we are showing 5 versions with this PR).

Can we figure it out that his happens automatically without any intervention from anybody.

Maybe this version bump could happen in scripts/update.sh (this is the hourly github action script) keeping website up-to-date.

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/tweag-nix-dev-update-46/26872/1

Manuals for all the version in the nix_versions.json files are
generated, but only the 3 previous are displayed on the website.

Includes a script, update-nix-versions.py, to update nix_versions.json.
New versions are automatically added. The script also updates the patch
version (e.g.: if latest 2.13 version is 2.13.3, the script will replace
that version with 2.13.4)
@jlesquembre
Copy link
Member Author

@garbas thanks for your feedback.
I added a script to automate version updates, called from scripts/update.sh as you suggested.
The version list is in the nix_versions.json file.
The UI is still the same as in the screenshot, there are 5 different versions. But documentation for more versions is generated. The goal is to prevent dead links when new nix versions are released.

Copy link
Contributor

@fricklerhandwerk fricklerhandwerk left a comment

Choose a reason for hiding this comment

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

LGTM

@garbas
Copy link
Member

garbas commented Apr 14, 2023

Sorry for taking long to review this, but I have some time planned to review this over the coming weekend.

@jlesquembre
Copy link
Member Author

@garbas no worries, thanks for reviewing it :)

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2023-06-05-documentation-team-meeting-notes-52/28937/1

@fricklerhandwerk
Copy link
Contributor

fricklerhandwerk commented Oct 4, 2023

It's very unfortunate how it turned out, but I'm closing this PR. @edolstra suggested to use build artifacts from Hydra instead of rebuilding the manual on this page. @garbas and the @NixOS/documentation-team agreed to move the manuals to nix.dev in the process of reworking nixos.org. @alejandrosame and me will sit down at Ocean Sprint (2023-10-16--20) and implement the manuals part, likely based on this PR. @jlesquembre thank you so much for your help regardless!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants