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

PathArc's name exposes implementation detail #9

Open
epage opened this issue Feb 11, 2018 · 4 comments
Open

PathArc's name exposes implementation detail #9

epage opened this issue Feb 11, 2018 · 4 comments

Comments

@epage
Copy link

epage commented Feb 11, 2018

This was first discussed on reddit.

For "low level" APIs, having Arc in the name makes sense. If this is a higher level API meant to focus more on the Python type of users, we should abstract that implementation detail and leave it out of the name

@vitiral
Copy link
Collaborator

vitiral commented Feb 11, 2018 via email

@vitiral
Copy link
Collaborator

vitiral commented Feb 11, 2018

Some possible names:

  • PathNonAbs
  • PathAny
  • PathMaybe

(I don't like any of these names personally, but thought it might get some ideas flowing)

@epage
Copy link
Author

epage commented Feb 12, 2018

Why not just Path? Its what you get when you strip the suffix. I assume the rest are Arcs too and are missing the suffix.

@vitiral
Copy link
Collaborator

vitiral commented Feb 12, 2018 via email

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

2 participants