Skip to content

Pedantry #98

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.

Sign up for GitHub

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Pedantry #98

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
6b2bf81
Changed lowrisc_pkgs -> lowriscPkgs
HU90m Apr 1, 2025
b7bb917
Changed llvm-cheriot identifier.
HU90m Apr 1, 2025
ecea895
Renamed python_ot to python-ot
HU90m Apr 1, 2025
16521b4
Renamed bazel_ot to bazelisk
HU90m Apr 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions flake.nix
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@
})
Copy link
Contributor

Choose a reason for hiding this comment

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

Re. the naming for let bindings, is there a standard somewhere that camel case is preferred? Or maybe just that camel > kebab > snake? I see all kinds of stuff. Maybe we could setup a lint if there is a standard somewhere

Copy link
Collaborator

Choose a reason for hiding this comment

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

https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md says use camel case for (non-package-name) variable names.

];
};
lowrisc_pkgs = import ./pkgs {inherit pkgs inputs;};
lowriscPkgs = import ./pkgs {inherit pkgs inputs;};
in {
checks = {
license = pkgs.stdenv.mkDerivation {
Expand All @@ -73,23 +73,23 @@
'';
};
};
packages = flake-utils.lib.filterPackages system lowrisc_pkgs;
packages = flake-utils.lib.filterPackages system lowriscPkgs;
devShells = {
opentitan = pkgs.callPackage ./dev/opentitan.nix {
inherit (lowrisc_pkgs) ncurses5-fhs ncurses6-fhs bazel_ot verilator_ot python_ot verible_ot;
inherit (lowriscPkgs) ncurses5-fhs ncurses6-fhs bazel_ot verilator_ot python_ot verible_ot;
};
cheriot = pkgs.mkShell {
Copy link
Contributor

Choose a reason for hiding this comment

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

Re. b7bb917, the commit message doesn't seem to match the changes? The change is just snake -> kebab

name = "cheriot";
packages =
(with lowrisc_pkgs; [llvm_cheriot xmake])
(with lowriscPkgs; [llvm_cheriot xmake])
++ (with pkgs; [
gnumake
magic-enum
srecord
]);
};
caliptra = pkgs.callPackage ./dev/caliptra.nix {
inherit (lowrisc_pkgs) verilator_caliptra riscv64-gcc_caliptra;
inherit (lowriscPkgs) verilator_caliptra riscv64-gcc_caliptra;
};
};
formatter = pkgs.alejandra;
Expand Down