-
Notifications
You must be signed in to change notification settings - Fork 10
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
Prepare packaging (zest) #24
Comments
Great job ! I have just one concern : does the buildout flow needs to break the pyramid flow ? (I've never used buildout) |
What are you calling pyramid flow !? |
I mean not having to learn buildout in a pyramid project. |
The only missing files are dev.ini and prod.ini which are generated now via buildout, i readded regenerated ones in b43dd69 |
It really looks like all those buildout boilerplate files don't belong to the project's main source code repository. Seeing things like "apache" in the source code raises a clear sign that this doesn't follow the SoC principle. Committing an empty Is there any added value in burying the source code under two levels of directories? I understand that creating the More generally speaking, what would buildout add to the project at this point? Buildout is great for managing non-Python dependencies, but if it's not used that way yet, what's the motivation for using it? My feeling is that buildout is too intrusive to be hardcoded in the project and I don't see at this point any benefit in introducing tight coupling between a deployment tool and the application source code. |
I ll not go in any debate just do no press the merge button if you dont understand the sense of sain commits. |
btw, i just subscribed out notifications, just speak to @leplatrem for anything else. |
Extracted from issue #17
The text was updated successfully, but these errors were encountered: