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

Allow storing config in (project/workspace)/.config/bacon.toml #268

Closed
joshka opened this issue Nov 21, 2024 · 5 comments · Fixed by #270
Closed

Allow storing config in (project/workspace)/.config/bacon.toml #268

joshka opened this issue Nov 21, 2024 · 5 comments · Fixed by #270

Comments

@joshka
Copy link
Contributor

joshka commented Nov 21, 2024

The rationale is https://dot-config.github.io/

Note this is not about XDG, but applying similar XDG-like logic to project folders to reduce the amount of cruft in the main repo tree.

Is this something you'd consider?

@Canop
Copy link
Owner

Canop commented Nov 21, 2024

Is this something you'd consider?

Yes. Not just right now, though, as I'm refactoring a lot to better support non-cargo tool chains.

@alerque
Copy link

alerque commented Nov 21, 2024

Yes, this should be an option.

Along similar lines, some projects also prefer merging tooling meta data as much as possible. My issue #241 is along those lines of allowing the config to be in the Cargo manifest. Now that Bacon also supports some other language ecosystems the same could also be done for pyproject.toml that can also house tooling configs.

@Canop
Copy link
Owner

Canop commented Dec 1, 2024

When we have both project-root/bacon.toml and project-root/.config/bacon.toml, which one should override the other one ?

Right now it looks like the .config one has more chance of being a personal one (and thus should override the other one), but I'm still unsure.

@Canop
Copy link
Owner

Canop commented Dec 1, 2024

Can you check #270 ?

@joshka
Copy link
Contributor Author

joshka commented Dec 1, 2024

When we have both project-root/bacon.toml and project-root/.config/bacon.toml, which one should override the other one ?

Right now it looks like the .config one has more chance of being a personal one (and thus should override the other one), but I'm still unsure.

I think you're right, but also one of those probably doesn't matter too much things, so either would be fine.

@Canop Canop closed this as completed in #270 Dec 2, 2024
@Canop Canop closed this as completed in dbb6f70 Dec 2, 2024
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 a pull request may close this issue.

3 participants