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

feat: allow extra attributes that are transparently passed through #32

Merged
merged 1 commit into from
Jun 20, 2022

Conversation

blaggacao
Copy link
Contributor

  • we might not know yet the downstream contracts that we might want
    to satisfy in order to hand down package dependencies of a particular
    nixago pepple (keeping the domain's integrity).
  • therefore, make the request module a freefrom module that can take
    untyped additional arguments of type anything

I tested in a downstream repo that the arguments are being passed along
as expected in a repl:

nix-repl> tf = treefmt "x86_64-linux"

nix-repl> tf.
tf.commands    tf.configData  tf.format      tf.output      tf.packages
nix-repl> tf.packages
[ «derivation /nix/store/qgag2xc1s01jvr3x43r91q1025lv2cg8-alejandra-1.4.0.drv» «derivation /nix/store/v9ry51pyilwdkclx3gqq7vymn685z3hg-prettier-2.7.0.drv» «derivation /nix/store/k92vnq4jz3cigsrmn6z1h5b41rpylplm-prettier-plugin-toml-0.3.1.drv» «derivation /nix/store/kmbx2xn033vcr4iaica1kg3l62n7par9-shfmt-3.5.1.drv» «derivation /nix/store/irj6ps94b4icsjihck42617w15ncygf9-treefmt-0.4.1.drv» ]

closes: #29

- we might not know yet the downstream contracts that we might want
  to satisfy in order to hand down package dependencies of a particular
  nixago pepple (keeping the domain's integrity).
- therefore, make the request module a freefrom module that can take
  untyped additional arguments of type anything

I tested in a downstream repo that the arguments are being passed along
as expected in a repl:

```
nix-repl> tf = treefmt "x86_64-linux"

nix-repl> tf.
tf.commands    tf.configData  tf.format      tf.output      tf.packages
nix-repl> tf.packages
[ «derivation /nix/store/qgag2xc1s01jvr3x43r91q1025lv2cg8-alejandra-1.4.0.drv» «derivation /nix/store/v9ry51pyilwdkclx3gqq7vymn685z3hg-prettier-2.7.0.drv» «derivation /nix/store/k92vnq4jz3cigsrmn6z1h5b41rpylplm-prettier-plugin-toml-0.3.1.drv» «derivation /nix/store/kmbx2xn033vcr4iaica1kg3l62n7par9-shfmt-3.5.1.drv» «derivation /nix/store/irj6ps94b4icsjihck42617w15ncygf9-treefmt-0.4.1.drv» ]
```

closes: nix-community#29
@jmgilman
Copy link
Collaborator

LGTM.

@jmgilman jmgilman merged commit 59843d3 into nix-community:master Jun 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow for an extension mechanism.
2 participants