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

When comparing, limit to facets in comparison #14

Open
wanderingstan opened this issue Aug 5, 2019 · 2 comments
Open

When comparing, limit to facets in comparison #14

wanderingstan opened this issue Aug 5, 2019 · 2 comments
Labels
question Further information is requested

Comments

@wanderingstan
Copy link
Contributor

Occurs to me that when when coworker shares a "diffenv" with me (and asks for me to compare with my env), we should by default run only the facets that were included in the original. This (1) helps with security and (2) allows the sharer to focus on the facets they're most interested in. E.g. if they didn't include the php facet, then neither should I, regardless of what's in my config.

A fancier version of the --share option could even pop open an editor (as in a git commit) to allow the sender to prever/edit their diffenv before sharing it.

@werg , thoughts?

@wanderingstan wanderingstan added the question Further information is requested label Aug 5, 2019
@werg
Copy link
Collaborator

werg commented Aug 5, 2019

Yes, I'd agree. If there's a repo config, should it take precedence?

@wanderingstan
Copy link
Contributor Author

wanderingstan commented Aug 8, 2019

My gut says that when doing a --compare, that the incoming env sets determines what we're comparing.

I suspect config file usage will be for power users, and if it was overriding passed-in files it would be quite confusing.

Comparing my env to yours will be the #1 use case, and there it makes the most sense that if I send you an env with my os,python, and git facets, then those are exactly the facets that I want to see compared with yours.

In this light, the config for a repo should be seen as the "suggested list of facets most relevant to this project", but once an env file has been generated, then it takes precedence.

But then, this is all just guessing. We need to get to dog-fooding this asap. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants