Hello. Anybody faced the issue with location deprecation problem? <https://github.com/ktorio/ktor/is...

# ktor

[sannysoft]

Hello. Anybody faced the issue with location deprecation problem?
https://github.com/ktorio/ktor/issues/688

[Deactivated User]

Didn’t notice. We recommend to use star imports as we do in Ktor itself (and I guess that’s why I didn’t had stumbled upon the issue). We use the kotlin standard, except using * imports not only for several imports after a threshold but even for single declaration imports.

In any case, I guess that issue will be fixed when we remove all the deprecations once 1.0 is reached

[Deactivated User]

Screenshot 2018-10-31 at 16.08.07.png

[sannysoft]

Great, thanks!!!

[lfielke]

Why does changing to * imports avoid hitting the deprecation error? The lower-case location has a Deprecated annotation, so I would have expected that using it either way would be an error

@Deprecated(
    "Use Location instead",
    replaceWith = ReplaceWith("Location"), level = DeprecationLevel.ERROR
)

[Deactivated User]

Maybe there is a resolution inconsistency there? Agree that one would expect to work the same in both cases

[Deactivated User]

I have explained the imports part here: https://ktor.io/quickstart/code-style.html

[lfielke]

Small grammar nitpick in that page... "The Official Coding Conventions doesn’t define what’s the recommended way of using imports." should be "The Official Coding Conventions don't define..."

[lfielke]

(because conventions is plural)