Fixes #38 - Adds configuration and models for postgres database

webcompat / webcompat-metrics-server

Server in charge of delivering different data to the webcompat-metrics-client

Mozilla Public License 2.0

6 stars 5 forks source link

Fixes #38 - Adds configuration and models for postgres database #40

Closed laghee closed 6 years ago

laghee commented 6 years ago

I closed #37 and added the config changes here since I couldn't get anything to run without adding models.py and initializing the SQLAlchemy object from there.

The paren changes were unintentional. I installed flake8, but maybe the formatter (autopep8) is wonky?

laghee commented 6 years ago

Travis wants the updated requirements.txt file from PR #36 ... I assume that means tests will pass if they're rerun after that pr is merged? If not, and I need to fix something, let me know!

laghee commented 6 years ago

@karlcow This is fairly rough in places due to some things in being in a tbd state (like exactly what data GH webhook issue-event payloads will include). Also, I had to get out of the javadoc mindset and transition into python ... and I am a person already suffering from overly-long-sentence disease. 😊

karlcow commented 6 years ago

Could you just rebase with the requirements.txt which has been merged before. Thanks.

laghee commented 6 years ago

OK... 🤞

karlcow commented 6 years ago

And it's a go! Thanks @laghee