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

HOT FIX: update README and remove isort.cfg #536

Merged
merged 1 commit into from
Jul 13, 2022
Merged

HOT FIX: update README and remove isort.cfg #536

merged 1 commit into from
Jul 13, 2022

Conversation

taylorfturner
Copy link
Contributor

@taylorfturner taylorfturner commented Jul 13, 2022

  • Removing isort.cfg. Setting now moved to setup.cfg
  • Also adding note in README about installing requirements-dev.txt

@taylorfturner taylorfturner added Bug Something isn't working Documentation Improvements or additions to documentation labels Jul 13, 2022
@taylorfturner taylorfturner self-assigned this Jul 13, 2022
@@ -1,4 +0,0 @@
[settings]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

now in setup.cfg

@@ -42,6 +42,7 @@ If the ML requirements are too strict (say, you don't want to install tensorflow

Install from pypi: `pip install DataProfiler`

If you want to install the dependencies for developing and updating the code base, be sure to run `pip install -r requirements-dev.txt`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

clarity in the README for installing additional dependencies when working on the repo

@micdavis micdavis enabled auto-merge (squash) July 13, 2022 16:49
@micdavis micdavis merged commit 04f5e13 into capitalone:main Jul 13, 2022
@taylorfturner taylorfturner deleted the hot_fix/remove_isort_config branch July 19, 2022 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants