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

[skip ci] implement new selector style #256

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

wolfv
Copy link
Member

@wolfv wolfv commented Mar 3, 2022

No description provided.

- cmake
- sel(unix): make
- if: unix
put: make
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just checking - if/put will be the new way to do it in the next boa release?

I'm happy to keep up with the changes and don't want backward compatibility concerns to slow down development - at this stage I think it's more important to experiment to find the right api.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, yeah, I am undecided. We're having some discussions with the conda community about this and I wanted to see how it feels. I think we should also have a shorthand - if(unix): make that we can easily normalize to the longer form.

This more verbose form adds a lot of "noise" to the recipes...

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would love to hear your thoughts. Are you on the conda slack? There is a spec discussion room that recently saw some activity!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll check out the slack 👍

The put did seem a bit verbose, but not too terrible!

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.

None yet

2 participants