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

tandoor-recipes: init at 1.4.1 #191532

Merged
merged 23 commits into from
Oct 3, 2022

Conversation

ambroisie
Copy link
Contributor

@ambroisie ambroisie commented Sep 16, 2022

Description of changes

Closes #145244.

A whole lot of python packages that were added to get this working. django-autocomplete-light has been marked doCheck = false to avoid having to package half a dozen more just for its tests.

The frontend does not make use of mkYarnPackage, because of the following issue.

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/)
  • 22.11 Release Notes (or backporting 22.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
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

django-js-reverse = super.django-js-reverse.overridePythonAttrs (_: {
doCheck = false;
});
Copy link
Contributor Author

Choose a reason for hiding this comment

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

See ierror/django-js-reverse#114 for when this isn't needed anymore.

@ambroisie ambroisie force-pushed the add-tandoor-recipes branch 4 times, most recently from 4d6bb88 to f3ce4f3 Compare September 16, 2022 18:07
@ofborg ofborg bot added the 8.has: package (new) This PR adds a new package label Sep 16, 2022
@ambroisie ambroisie force-pushed the add-tandoor-recipes branch 2 times, most recently from e90189a to 788d704 Compare September 17, 2022 15:48
@nixos-discourse
Copy link

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

https://discourse.nixos.org/t/prs-ready-for-review/3032/1196

pkgs/development/python-modules/pyrdfa3/default.nix Outdated Show resolved Hide resolved
pkgs/development/python-modules/microdata/default.nix Outdated Show resolved Hide resolved
pkgs/development/python-modules/microdata/default.nix Outdated Show resolved Hide resolved
pkgs/applications/misc/tandoor-recipes/default.nix Outdated Show resolved Hide resolved
pkgs/applications/misc/tandoor-recipes/default.nix Outdated Show resolved Hide resolved
pkgs/applications/misc/tandoor-recipes/default.nix Outdated Show resolved Hide resolved
pkgs/applications/misc/tandoor-recipes/default.nix Outdated Show resolved Hide resolved
pkgs/applications/misc/tandoor-recipes/common.nix Outdated Show resolved Hide resolved
@ambroisie ambroisie force-pushed the add-tandoor-recipes branch 4 times, most recently from 963107b to 08eb42f Compare September 17, 2022 19:39
@ambroisie
Copy link
Contributor Author

ambroisie commented Sep 17, 2022

@SuperSandro2000 addressed all your comments.

@ambroisie ambroisie force-pushed the add-tandoor-recipes branch 2 times, most recently from c0df50e to a95227f Compare September 17, 2022 20:18
@ambroisie ambroisie force-pushed the add-tandoor-recipes branch 3 times, most recently from 950560f to 25d511d Compare September 29, 2022 16:27
@bobby285271 bobby285271 added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Oct 1, 2022
@ambroisie ambroisie force-pushed the add-tandoor-recipes branch from 25d511d to 83f5c2d Compare October 3, 2022 07:49
@ofborg ofborg bot requested a review from SuperSandro2000 October 3, 2022 07:58
@bobby285271 bobby285271 removed the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Oct 3, 2022
@SuperSandro2000 SuperSandro2000 merged commit 1385382 into NixOS:master Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 6.topic: python 8.has: changelog 8.has: documentation This PR adds or changes documentation 8.has: module (update) This PR changes an existing module in `nixos/` 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 11-100 10.rebuild-linux: 101-500
Projects
None yet
Development

Successfully merging this pull request may close these issues.

package/module request: tandoor recipes
7 participants