Skip to content

Commit

Permalink
WIP
Browse files Browse the repository at this point in the history
  • Loading branch information
yannham committed Oct 1, 2024
1 parent 13fe6cc commit 76f64b4
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 113 deletions.
4 changes: 2 additions & 2 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -101,13 +101,13 @@ metrics = "0.21"
metrics-util = "0.15"

topiary-core = "0.4.0"
topiary-queries = { version = "0.4.2", default-features = false, features = ["nickel"] }
topiary-queries = { version = "=0.4.2", default-features = false, features = ["nickel"] }
# This version should agree with the topiary-query version: Nickel queries
# target a specific version of the Nickel grammar (though that dependency
# doesn't appear explicitly in `topiary-queries`'s Cargo.toml file). For now you
# might have to look at Topiary's commit messages, but in a near future, we'll
# try to make this data more accessible, e.g. in the query file
tree-sitter-nickel = "0.2.0"
tree-sitter-nickel = "=0.2.0"
tempfile = "3.5.0"

[profile.dev.package.lalrpop]
Expand Down
92 changes: 1 addition & 91 deletions flake.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

22 changes: 4 additions & 18 deletions flake.nix
Original file line number Diff line number Diff line change
Expand Up @@ -13,22 +13,6 @@
crane = {
url = "github:ipetkov/crane";
};
topiary = {
# We pin this specific PR because we need a recent version of Topiary to
# get rid of the dreaded bash version drift
# (https://github.com/tweag/topiary/issues/702), but current the latest
# Topiary doesn't build with Nix since the addition of dynamic loading of
# grammars. So once https://github.com/tweag/topiary/pull/747
# (allow-path-for-grammar-source) is merged, we can go back to following
# the main branch.
url = "github:tweag/topiary/tmp-allow-for-path-grammar-source";
inputs = {
nixpkgs.follows = "nixpkgs";
crane.follows = "crane";
flake-utils.follows = "flake-utils";
rust-overlay.follows = "rust-overlay";
};
};
nix-input = {
url = "github:nixos/nix";
inputs = {
Expand All @@ -50,7 +34,6 @@
, pre-commit-hooks
, rust-overlay
, crane
, topiary
, nix-input
}:
let
Expand Down Expand Up @@ -175,6 +158,10 @@
];
};

# We could use Topiary here, but the Topiary version pulled from Nix
# and the one baked in Nickel could differ. It's saner that what we
# check in the CI is matching exactly the formatting performed by the
# `nickel` binary of this repo.
nickel-format = {
name = "nickel-format";
description = "The nickel formatter";
Expand Down Expand Up @@ -511,7 +498,6 @@
pkgs.yarn2nix
pkgs.nodePackages.markdownlint-cli
pkgs.python3
topiary.packages.${system}.topiary-cli-nix
];

shellHook = (pre-commit-builder { inherit rust; checkFormat = true; }).shellHook + ''
Expand Down

0 comments on commit 76f64b4

Please sign in to comment.