Skip to content
This repository has been archived by the owner on Oct 1, 2019. It is now read-only.

Switch to scoped @prettier package #16

Merged
merged 1 commit into from Dec 30, 2017
Merged

Switch to scoped @prettier package #16

merged 1 commit into from Dec 30, 2017

Conversation

azz
Copy link
Member

@azz azz commented Dec 29, 2017

Just published current code as 0.0.0-development - https://www.npmjs.com/package/@prettier/python

@taion
Copy link
Contributor

taion commented Dec 29, 2017

Namespacing, neat!

@j-f1
Copy link
Member

j-f1 commented Dec 29, 2017

Namespaces are one honking great idea -- let's do more of those!

From the zen of Python. (official link)

@taion
Copy link
Contributor

taion commented Dec 29, 2017

Yes that's actually sort of a lie (in the Python context). Namespace packages in Python are a dumpster fire and in general people who have used them move away from using them. They're very much "0/10 would not recommend" there.

It's not anybody's fault or anything – it's just the way Python package resolution works, because the package name for importing doesn't connect in any necessary way to the package name on PyPI. Makes hostile forks easier, though.

@taion
Copy link
Contributor

taion commented Dec 29, 2017

(They're great in the npm ecosystem, though!)

@azz azz merged commit a2ef27e into prettier:master Dec 30, 2017
@azz azz deleted the scoped branch December 30, 2017 00:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants