Skip to content

Update circle 2 #338

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

Merged
merged 5 commits into from
Aug 17, 2018
Merged

Update circle 2 #338

merged 5 commits into from
Aug 17, 2018

Conversation

T4rk1n
Copy link
Contributor

@T4rk1n T4rk1n commented Aug 16, 2018

  • Update the circle configs to use version 2.
  • Removed tox.
  • Add python 3.7 test environment.

It's mostly the same as #222 but with python 3.7 tests. Pylint for python 3.7 has changed some rules naming so I created another .pylintrc for that one.

T4rk1n added 5 commits August 14, 2018 12:26
Try fix py37 pylint compat.

Try fix py37 pylint compat.

Ignore the renamed/new pylint rules.
Remove tox.ini
@T4rk1n T4rk1n requested review from chriddyp and rmarren1 August 16, 2018 18:41
PYLINTRC: .pylintrc

"python-3.7":
<<: *test-template
Copy link
Member

Choose a reason for hiding this comment

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

nice work getting this test-template stuff in. code is super DRY 🐫

@chriddyp
Copy link
Member

Looks great @T4rk1n ! There might be some CONTRIBUTING.md files or something that need updating as well. Otherwise, 💃

@T4rk1n
Copy link
Contributor Author

T4rk1n commented Aug 17, 2018

@chriddyp There is no CONTRIBUTING.md for dash, maybe we should add one ?

@T4rk1n T4rk1n merged commit 5fc998a into master Aug 17, 2018
@T4rk1n T4rk1n deleted the update-circle-2 branch August 17, 2018 20:31
HammadTheOne pushed a commit to HammadTheOne/dash that referenced this pull request May 28, 2021
HammadTheOne pushed a commit that referenced this pull request Jul 23, 2021
AnnMarieW pushed a commit to AnnMarieW/dash that referenced this pull request Jan 6, 2022
Change `React.PropTypes` to just `PropTypes` in `Tabs.children`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants