Skip to content
This repository was archived by the owner on Feb 26, 2021. It is now read-only.

Fixing context when calling djangofy #30

Merged
merged 2 commits into from
Oct 3, 2017
Merged

Fixing context when calling djangofy #30

merged 2 commits into from
Oct 3, 2017

Conversation

cstrap
Copy link
Owner

@cstrap cstrap commented Oct 3, 2017

No description provided.

Copy link

@lintly-bot lintly-bot left a comment

Choose a reason for hiding this comment

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

Lintly has detected code quality issues in this pull request.

@@ -43,19 +43,23 @@ def install_cli():

@staticmethod
def project_setup(project):
click.echo(click.style('running `vue init webpack {project}`'.format(project=project), fg='yellow'))

Choose a reason for hiding this comment

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

E501: line too long (108 > 79 characters)

@@ -132,8 +131,7 @@ def test_djstartvueapp_django_ok(self, mock_djangofy):
result = self.runner.invoke(cli.cli, ['djstartvueapp', 'myapp'])
# Then
mock_vuejsbuilder.assert_called_once()
mock_djangofy.assert_called_once()
self.assertEqual('Creating myapp\n', result.output)
self.assertEqual('Creating myapp\nMaking Vue.js myapp into django app\n', result.output)

Choose a reason for hiding this comment

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

E501: line too long (104 > 79 characters)

@cstrap cstrap merged commit ef3a0cd into master Oct 3, 2017
@coveralls
Copy link

coveralls commented Oct 3, 2017

Coverage Status

Coverage decreased (-2.1%) to 85.821% when pulling af1a6a3 on develop into 7634726 on master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants