Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Incorporate ./shells #38

Closed
blaggacao opened this issue Dec 24, 2020 · 4 comments
Closed

Incorporate ./shells #38

blaggacao opened this issue Dec 24, 2020 · 4 comments
Labels
enhancement New feature or request

Comments

@blaggacao
Copy link
Contributor

blaggacao commented Dec 24, 2020

Is your feature request related to a problem? Please describe.
I think it is time to share efforts in building user provided dev environments. I also think this template repo leverages the flakes model of sharing to the most elaborate extend. I would be pleased if I could use something like github.com:user/nixflk#devShells.rust of a trusted user as an input to my own variety of devshells.

Describe the solution you'd like
I would like such sharing to evolve around https://github.com/numtide/devshell to provide an advanced and consistent devshell environment.

Describe alternatives you've considered
none — inmature thought.

Additional context


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@nrdxp
Copy link
Collaborator

nrdxp commented Dec 25, 2020

I like the idea of providing various shells as an output. I've had a similar idea recently. I think the simplest way to go about this would be to add a custom flake output attribute. Something like shells. We will probably have to write some checks as well.

@blaggacao
Copy link
Contributor Author

I hope @zimbatm could add to the thought-noodle-soop we are cooking here. Should we add more salt? A shells attribute is already quite spicey. But will it be allowed once / if flakes outputes will be typed?

blaggacao pushed a commit to blaggacao/devos that referenced this issue Dec 29, 2020
blaggacao pushed a commit to blaggacao/devos that referenced this issue Dec 29, 2020
blaggacao pushed a commit to blaggacao/devos that referenced this issue Dec 29, 2020
@nrdxp
Copy link
Collaborator

nrdxp commented Dec 29, 2020

#54 is a step in this direction.

blaggacao pushed a commit to blaggacao/devos that referenced this issue Dec 29, 2020
@blaggacao
Copy link
Contributor Author

I try to second: #55

@nrdxp nrdxp added the enhancement New feature or request label Dec 30, 2020
This was referenced Feb 15, 2021
Pacman99 added a commit that referenced this issue Feb 26, 2022
38: importers: filter out empty attrsets r=blaggacao a=Pacman99

Allows you to have arbitrary folders that only get picked up if .nix files are in there. This could be useful for a folder dedicated to contains patches. And generally improves safety of the now fully recursive importers.

I decided to put this in rakeLeaves, since it makes no sense to rake pointless folders and the change propagates to all importers.

Co-authored-by: Pacman99 <pachum99@myrdd.info>

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants