-
Notifications
You must be signed in to change notification settings - Fork 106
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
[12.0][PORT] #622 port from 10.0: force technical user recompute export #677
Conversation
@HviorForgeFlow We use MQT_PIP for tests in shopinvader (it can be activated for OCA too at demand). So, you don't need to change oca_dependencies.txt as it wouldn't be taken into account as travis uses pip install -r test-requirements.txt in that case. More flexible. |
827951a
to
6f5b52e
Compare
I can't guess how to add to addons in a PR inside test-requirements.txt Update: I think I found the way :) Update 2: Something is still missing :( |
781809e
to
f1635a2
Compare
Codecov Report
@@ Coverage Diff @@
## 12.0 #677 +/- ##
==========================================
- Coverage 91.09% 91.01% -0.08%
==========================================
Files 137 137
Lines 3997 3997
==========================================
- Hits 3641 3638 -3
- Misses 356 359 +3
Continue to review full report at Codecov.
|
780c644
to
65634b4
Compare
@HviorForgeFlow Can you remove test requirement line and rebase ? |
…r backend view) [ADD] Add unit tests
65634b4
to
b576919
Compare
/ocabot merge minor |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 7e50510. Thanks a lot for contributing to shopinvader. ❤️ |
Port #622 from 10.0 to 12.0