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.

Sign up for GitHub

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

Jump to bottom

Investigate modules from app.py #76

Closed
mariekers opened this issue Mar 18, 2020 · 0 comments · Fixed by #55
Closed

Investigate modules from app.py #76

mariekers opened this issue Mar 18, 2020 · 0 comments · Fixed by #55
Assignees
@quinn-dougherty @sam-writer
Labels
documentation Improvements or additions to documentation enhancement New feature or request python
Milestone
modularize python...

Comments

@mariekers
Copy link
Contributor

how can we break app.py into modules?

  • requirement: put custom set variables in separate scripts so partners can maintain that module themselves while updating the rest of the app from the repo

Done: app.py modules understood and documented, plan for re-factoring exists
Quinn & Sam - wanna make a braintrust?
@mariekers mariekers added this to the modularize python script milestone Mar 18, 2020
@mariekers mariekers mentioned this issue Mar 18, 2020
Closed
@mariekers mariekers added documentation Improvements or additions to documentation enhancement New feature or request python labels Mar 18, 2020
@sam-writer sam-writer mentioned this issue Mar 19, 2020
Merged
kochizufan pushed a commit to code4history/chime-1 that referenced this issue Mar 22, 2020
@cjbayesian
Merge pull request CodeForPhilly#76 from pennsignals/cfp_latest
b720908 
Cfp latest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@quinn-dougherty quinn-dougherty

@sam-writer sam-writer

Labels
documentation Improvements or additions to documentation enhancement New feature or request python
Projects
None yet
Milestone
modularize python script
Development

Successfully merging a pull request may close this issue.

Refactor (move towards module) CodeForPhilly/chime
3 participants
@mariekers @quinn-dougherty @sam-writer