-
Notifications
You must be signed in to change notification settings - Fork 29
Clean: update references to modes prod and demo #74
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
Conversation
84556db
to
fbe871e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good but please remove change on RUNNING_ENV.
Also I'm not sure what CLN is your commit message stands for. To me it means like Comitato di Liberazione Nazionale 😄
README.rst
Outdated
ignore_if: test "${RUNNING_ENV}" != "prod" | ||
ignore_if: test "${RUNNING_ENV}" != "full" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's MARABUNTA_MODE that is either full or sample.
marabunta/parser.py
Outdated
ignore_if: test "${RUNNING_ENV}" != "prod" | ||
ignore_if: test "${RUNNING_ENV}" != "full" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
Thanks for the review. PR updated (unless the commit name). Once the current change is ok for you I will push force with:
|
These two modes where deprecated in favor of: prod->full,demo->sample - full: mode that will launch scripts that are dependent to production data (not launcher by ci - sample: these are sample related to the customers project. We dont want to mixup with odoo demo data
641607f
to
bdcc491
Compare
These two modes where deprecated in favor of: prod->full,demo->sample