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.

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

Fix #151 setup pre-commit and black #173

Merged
merged 2 commits into from
Apr 22, 2019
Merged

Fix #151 setup pre-commit and black #173

merged 2 commits into from
Apr 22, 2019

Conversation

rochacbruno
Copy link
Member

developers should now do:

make all

To ensure pre-commits are installed and running.

Optionally:

make clean instal
make run-pre-commit
make test
make test_examples
make coverage-report

@rochacbruno rochacbruno added enhancement Not a Bug Not a Problem, expected behavior labels Apr 22, 2019
@rochacbruno rochacbruno added this to the 2.0.1 (pre-release) milestone Apr 22, 2019
@codecov
Copy link

codecov bot commented Apr 22, 2019

Codecov Report

Merging #173 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #173   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          18     18           
  Lines         700    705    +5     
=====================================
+ Hits          700    705    +5
Impacted Files Coverage Δ
dynaconf/validator_conditions.py 100% <ø> (ø) ⬆️
dynaconf/utils/boxing.py 100% <ø> (ø) ⬆️
dynaconf/utils/files.py 100% <100%> (ø) ⬆️
dynaconf/loaders/py_loader.py 100% <100%> (ø) ⬆️
dynaconf/utils/parse_conf.py 100% <100%> (ø) ⬆️
dynaconf/loaders/env_loader.py 100% <100%> (ø) ⬆️
dynaconf/contrib/django_dynaconf_v2.py 100% <100%> (ø) ⬆️
dynaconf/contrib/flask_dynaconf.py 100% <100%> (ø) ⬆️
dynaconf/loaders/base.py 100% <100%> (ø) ⬆️
dynaconf/loaders/yaml_loader.py 100% <100%> (ø) ⬆️
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a2ae3d...22d3dc6. Read the comment docs.

@rochacbruno rochacbruno merged commit e485217 into master Apr 22, 2019
@rochacbruno rochacbruno deleted the black branch April 22, 2019 18:08
rochacbruno added a commit that referenced this pull request Apr 22, 2019
Shortlog of commits since last release:

    Bruno Rocha (17):
          Release version 2.0.0
          Added Django explicit mode to docs (#149)
          HOTIX: Django doc
          Logger is now cached (removed logging import time overhead)
          Update issue templates
          Adjusts issue templates
          Fix Typo in issue template
          fix #160 - invoking directory should not be search breaking point.
          Add explicit call to main() on cli.py (#165)
          Generate coverage.xml file (#167)
          Fix #166 renamed GLOBAL_ENV_ to ENVVAR_PREFIX_ (#168)
          Fix #169 renamed SETTINGS_MODULE_ to SETTINGS_FILE_ (#170)
          HOTFIX config.md on docs [skip ci] (#171)
          Fix some open file descriptors on exampls and tests (#172)
          Fix #151 setup pre-commit and black (#173)
          Add CONTRIBUTING.md, conrtib isntructions and Black badge (#174)
          Fix release script

    David Moreau Simard (1):
          Fix typos in bash export examples

    Jan Willhaus (2):
          Skip reloading envs for validators that only apply to current_env (#162)
          Fix #163 Allow disabling env prefix (#164)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Not a Bug Not a Problem, expected behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant