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

Small cleanups for various editors. Play nice with built in test-runners #121

Merged
merged 1 commit into from
Dec 15, 2016
Merged

Small cleanups for various editors. Play nice with built in test-runners #121

merged 1 commit into from
Dec 15, 2016

Conversation

t8y8
Copy link
Contributor

@t8y8 t8y8 commented Dec 15, 2016

  1. Add Sublime and VSCode settings files to the gitignore
  2. The init in the test folder makes a lot of test runners angry because it imports some packages it doesn't need to. By removing that I can make some ides happier. This has no effect on setup.py test

Copy link

@jz-huang jz-huang left a comment

Choose a reason for hiding this comment

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

looks good =)

@jz-huang
Copy link

🚀

@t8y8 t8y8 merged commit cdccd2f into tableau:development Dec 15, 2016
@t8y8 t8y8 deleted the minor-touchup branch December 15, 2016 02:10
graysonarts pushed a commit that referenced this pull request Jan 11, 2017
* Fix #117 by only attempting files with the right extension inside the archive (#118)

* Commenting and Docstring cleanup. A few very small code cleanups (#120)

Add docstrings and remove clutter. I also made some very tiny tweaks to some code for clarity.

* Small cleanups for various editors. Play nice with built in test-runners (#121)

* Add Py36, update travis to use pycodestyle (#124)

* Add `initial sql` and `query band` support (#123)

Addresses #109 and #110

* Prep for release of 0.6 (#125)

* Prep for release of 0.6

* wordsmithing the changelog
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