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

doc/contributing: update build instructions after nrd #245358

Merged
merged 1 commit into from
Jul 29, 2023

Conversation

asymmetric
Copy link
Contributor

@asymmetric asymmetric commented Jul 25, 2023

We have moved away from Make since nixos-render-docs was introduced to
build the nixpkgs manual in #239636.

Description of changes
Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.11 Release Notes (or backporting 23.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

@asymmetric asymmetric requested a review from pennae July 25, 2023 10:21
@github-actions github-actions bot added the 8.has: documentation This PR adds or changes documentation label Jul 25, 2023
@asymmetric asymmetric force-pushed the nixpkgs-doc-build-instructions branch 4 times, most recently from 34ed590 to e49e3f8 Compare July 25, 2023 10:36
@asymmetric asymmetric changed the title doc/contributing: update build instructions doc/contributing: update build instructions after nrd Jul 25, 2023
@asymmetric asymmetric force-pushed the nixpkgs-doc-build-instructions branch from e49e3f8 to 3d303fa Compare July 25, 2023 10:38
@asymmetric asymmetric added the 6.topic: documentation Meta-discussion about documentation and its workflow label Jul 25, 2023
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux labels Jul 25, 2023
@asymmetric asymmetric force-pushed the nixpkgs-doc-build-instructions branch from 3d303fa to 5d9e5ef Compare July 25, 2023 13:26
We have moved away from Make since nixos-render-docs was introduced to
build the nixpkgs manual in NixOS#239636.
@asymmetric asymmetric force-pushed the nixpkgs-doc-build-instructions branch from 5d9e5ef to 35471bd Compare July 25, 2023 13:26

If you experience problems, run `make debug` to help understand the docbook errors.

After making modifications to the manual, it's important to build it before committing. You can do that as follows:
Copy link
Contributor

Choose a reason for hiding this comment

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

just notice that this message went away. not sure how necessary it is anymore since markdown literally does not have a concept of invalid syntax (and egregious nrd-extension errors would be caught by CI).

@pennae pennae merged commit dcd5591 into NixOS:master Jul 29, 2023
@asymmetric asymmetric deleted the nixpkgs-doc-build-instructions branch July 30, 2023 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: documentation Meta-discussion about documentation and its workflow 8.has: documentation This PR adds or changes documentation 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants