-
Notifications
You must be signed in to change notification settings - Fork 476
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
[alias] attribute #2349
Comments
I like the idea of keeping all parts of a recipe in one place. Especially when considering modules, etc. |
@casey I'm not sure if you had a chance to fully review the original PR, but my first commit is my first attempt in which I tried to use the I wasn't liking how the code turned out, and felt like I was fighting Just to implement it. That's why I moved to using Identifiers/Names/Tokens (not sure which is the most technically correct qualifier here 😄), and because aliases are already processed that way. I've made a couple more attempts at implementing it with strings, but haven't been successful. Would love to make another attempt and get this feature eventually merged, but if you could possibly point me in the right direction for implementing this with strings (if you've got any ideas for it) that would be super helpful. |
I'm still honestly not totally sold on this feature, since it feels like it just duplicates the existing way of creating aliases. |
PR #2344 added a new attribute,
[alias]
providing an alternative way for declaring recipe aliases. I personally lean towards not adding a new attribute, but I don't feel super strongly, so if it's something that people wanted, I wouldn't mind adding it.I think ideally, would take a string literal:
[alias: "foo"]
, which would initially be constrained to be a valid identifier. This could be relaxed in the future to allow non-identifier aliases.See the original PR for some of the discussion.
The text was updated successfully, but these errors were encountered: