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

PR: Enable CircleCI and first tests addition #30

Merged
merged 21 commits into from Aug 19, 2017
Merged

Conversation

andfoy
Copy link
Member

@andfoy andfoy commented Aug 18, 2017

Fixes #24
Fixes #27

@andfoy andfoy added this to the v0.1 milestone Aug 18, 2017
@andfoy andfoy self-assigned this Aug 18, 2017
circle.yml Outdated
# The last container is used to test with pyqt5 wheels
PY_VERSIONS: "2.7 3.6 3.5"
# For Coveralls
COVERALLS_REPO_TOKEN: sTGxv5iQxuiBQCoxLB53t6m2xSe58rAIq
Copy link
Member

@rlaverde rlaverde Aug 18, 2017

Choose a reason for hiding this comment

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

This token should be deleted from here (and regenerated), and added instead to https://circleci.com/gh/spyder-ide/spyder-vim/edit#env-vars

.checkignore Outdated
@@ -0,0 +1,16 @@
# Quantified code ignore file
Copy link
Member

Choose a reason for hiding this comment

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

Please remove this file. Quantified Code was closed.

Copy link
Member

@rlaverde rlaverde Aug 18, 2017

Choose a reason for hiding this comment

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

Maybe we could use Scrutinizer in the plugins as qtpy does (It's somehow similar)

@spyder-ide spyder-ide deleted a comment from coveralls Aug 18, 2017
@spyder-ide spyder-ide deleted a comment from coveralls Aug 18, 2017
@spyder-ide spyder-ide deleted a comment from coveralls Aug 18, 2017
@ccordoba12
Copy link
Member

Now you need to fix all style issue reported by ciocheck ;-)

setup.py Outdated

setup(
name='spyder.vim',
version="0.1.0",
name='spyder_vim',
Copy link
Member

Choose a reason for hiding this comment

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

This needs to be spyder-vim, not spyder_vim

Copy link
Member Author

Choose a reason for hiding this comment

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

Then I should also update spyder-terminal then!

Copy link
Member

Choose a reason for hiding this comment

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

Please open an issue about it before doing it. I don't know if it's possible at this point.

Copy link
Member Author

Choose a reason for hiding this comment

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

It seems that the package is right: https://pypi.python.org/pypi/spyder-terminal

Copy link
Member

Choose a reason for hiding this comment

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

Great! Then it's probably safe to do it ;-)

setup.py Outdated
packages=find_packages(exclude=['contrib', 'docs', 'tests*']),
keywords=["spyder ide plugin addon vi vim"],
url='https://github.com/Nodd/spyderplugins.vim',
keywords=['Spyder', 'Plugin'],
Copy link
Member

Choose a reason for hiding this comment

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

Vim is missing here.

@andfoy andfoy merged commit 9819e58 into master Aug 19, 2017
@andfoy andfoy deleted the ci_tests_integration branch August 19, 2017 01:50
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.

None yet

3 participants