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

Use actual namespace for kscript annotations #80

Closed
holgerbrandl opened this issue Dec 8, 2017 · 2 comments
Closed

Use actual namespace for kscript annotations #80

holgerbrandl opened this issue Dec 8, 2017 · 2 comments
Assignees

Comments

@holgerbrandl
Copy link
Collaborator

Currently, kscript annotations live in the default package, so that users don't need to import them when writing scripts (which typically don't declare a package).

But the default package has issues and is considered bad practice https://youtrack.jetbrains.com/issue/KT-21422

So we may want to simply reimplement the annotations in an actual namespace (kscript.annotations) for better design and tooling within IJ.

@holgerbrandl holgerbrandl self-assigned this Dec 8, 2017
@holgerbrandl holgerbrandl changed the title Provide annotations in default and actual namespace Use actual namespace for kscript annotations Dec 12, 2017
@holgerbrandl
Copy link
Collaborator Author

Won't fix for now, but let's keep it open for discussion.

So far the only usecase that would benefit from an actual namespace are scripts that live in packages themselfes. See #81 for an example

@holgerbrandl
Copy link
Collaborator Author

Closed, since it is unlikely that we can resolve it anytime. It's a design flaw in favor of a more streamlined usage (less typing).

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

No branches or pull requests

1 participant