Skip to content
This repository was archived by the owner on Apr 2, 2021. It is now read-only.

WIP: tox migration - #45

Open
ripoul wants to merge 14 commits into
masterfrom
develop
Open

WIP: tox migration#45
ripoul wants to merge 14 commits into
masterfrom
develop

Conversation

@ripoul

@ripoul ripoul commented Oct 23, 2019

Copy link
Copy Markdown
Owner

first refactor

@ripoul
ripoul requested a review from TristanElain as a code owner October 23, 2019 12:37
@ripoul ripoul changed the title Refactor all code WIP: Refactor all code Oct 23, 2019
@ripoul ripoul changed the title WIP: Refactor all code WIP: tox migration Oct 23, 2019
@ripoul
ripoul requested a review from gearsof44 October 23, 2019 12:48
@codecov

codecov Bot commented Oct 23, 2019

Copy link
Copy Markdown

Codecov Report

Merging #45 into master will increase coverage by 12.79%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master      #45       +/-   ##
===========================================
+ Coverage   46.47%   59.26%   +12.79%     
===========================================
  Files          12       17        +5     
  Lines         284      356       +72     
===========================================
+ Hits          132      211       +79     
+ Misses        152      145        -7

@ripoul

ripoul commented Oct 23, 2019

Copy link
Copy Markdown
Owner Author

@gearsof44 tu peux add tox au requierements

Comment thread apps/georidePosition/settings.py Outdated
Comment thread develop_requirements.txt Outdated
Comment thread requirements.txt Outdated
@ripoul

ripoul commented Oct 24, 2019

Copy link
Copy Markdown
Owner Author

@gearsof44 You need GEORIDE_EMAIL and GEORIDE_PASSWORD as env vars for the test to pass ? I can set them in the CI as secret. But External PR will not work...

@ripoul

ripoul commented Oct 24, 2019

Copy link
Copy Markdown
Owner Author

@all-contributors please add @gearsof44 for code, test

@allcontributors

Copy link
Copy Markdown
Contributor

@ripoul

I've put up a pull request to add @gearsof44! 🎉

@ripoul

ripoul commented Oct 24, 2019

Copy link
Copy Markdown
Owner Author

@all-contributors please add @gearsof44 for code, test

@allcontributors

Copy link
Copy Markdown
Contributor

@ripoul

I've put up a pull request to add @gearsof44! 🎉

Comment thread apps/map/tests/fixtures.py Outdated
@ripoul

ripoul commented Oct 24, 2019

Copy link
Copy Markdown
Owner Author

@gearsof44 why 2 requierement file ?

@ripoul

ripoul commented Oct 24, 2019

Copy link
Copy Markdown
Owner Author

@gearsof44 do not force 80% codecov : we have a bot who provide us a coverage report on each PR. For me it is enougth to manage those problems.
If an external PR is made it will confuse the user.

gguillet-rm and others added 2 commits October 24, 2019 20:14
…tor model user & profile to app accounts, add app services containing all georide services, add georide driver + unauthenticated georide driver, begin tests for unauthenticated georide driver
Comment thread app.yaml
Comment on lines 1 to 3
runtime: python37
entrypoint: gunicorn -b :$PORT georidePosition.wsgi
entrypoint: gunicorn -b :$PORT apss.georide_position.wsgi

@ripoul ripoul Nov 21, 2019

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

apss -> you mean apps

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants