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

clarify module names and locations #29

Open
seb-pereira opened this issue Sep 5, 2014 · 5 comments
Open

clarify module names and locations #29

seb-pereira opened this issue Sep 5, 2014 · 5 comments
Assignees
Labels

Comments

@seb-pereira
Copy link
Member

No description provided.

@seb-pereira seb-pereira self-assigned this Sep 5, 2014
@seb-pereira seb-pereira added this to the 0.3.0 milestone Sep 5, 2014
@seb-pereira seb-pereira changed the title Rename folder handler to src Rename folder handlers to src Sep 5, 2014
@cjolif
Copy link
Contributor

cjolif commented Sep 5, 2014

what is the rationale?

@seb-pereira
Copy link
Member Author

Because is doesn't really contain only handlers.

@cjolif
Copy link
Contributor

cjolif commented Sep 5, 2014

ok, my issue is that we do have "sources" (src) outside of this folder as well right? So this is misleading to me. What about alternatively either:

  • moving the modules upwards to the root removing the need for sub directory
  • renaming it "utils"? or "internal"?

@seb-pereira
Copy link
Member Author

Actually there is no such "src" folder. Also, now you ask the question I am not sure what would be the best way to organize modules because I have some ongoing refactoring as part of the release 0.3.0 that could impact the choice of file names/locations... So I think I will remove the milestone for this issue and just call it "clarify module names and locations" (anyway, this is not a high priority...). What do think?

@cjolif
Copy link
Contributor

cjolif commented Sep 5, 2014

+1

@seb-pereira seb-pereira removed this from the 0.3.0 milestone Sep 5, 2014
@seb-pereira seb-pereira changed the title Rename folder handlers to src clarify module names and locations Sep 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants