-
-
Notifications
You must be signed in to change notification settings - Fork 655
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 qualified metadata names #3959
Comments
+1 , we need something that scales with the growing number of targets and target-specific metadata. |
+1 and maybe we should move compiler internal metadata into it's own namespace before 3.2 |
What is the different between the proposal and using prefix (e.g. "@:jsSomeName")? |
|
|
I'm not sure about it : we have a lot of metas which are for instance cs+java, and I'm not sure how we should handle the transition from global metadata to namespaced one. Or would that just be a filter ? for instance @:js.meta would be the same as @:meta but would check that the metadata is available in JS ? |
I'm not sure about the transition either but I really feel like we should enforce some naming schemes. It's getting a bit out of control. |
I strongly agree that our metadata naming is getting out of control. It's not only untyped and unchecked (so e.g. a misspell makes the behaviour silently fail), but it lacks namespacing. Many macro libraries for example use the Typed metas can always be an option :)) |
+1 for typed metas! |
I think it would make sense. We all agree that strongly typed code is better than untyped - so I see no reason why a metadata should be untyped. We could maybe still allow the untyped metas as well - implementation still needs to be defined. |
Alright, let's 3.3 the discussion... |
Following a discussion we had on #haxe: It would be nice if we could use
@:js.someName
in metadata to group them. Internally this could just become a metadata named:js.someName
.Our metadata is growing into a bit of a mess so I would like to address this sooner rather than later. Any objections?
The text was updated successfully, but these errors were encountered: