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

ref: Reformat and pre-commit configuration #7344

Merged
merged 2 commits into from
Oct 9, 2020
Merged

Conversation

iamareebjamal
Copy link
Member

No description provided.

@@ -22,7 +22,8 @@
def test_delete_ticket_holder_created_currently(db):
"""Method to test not deleting ticket holders with no order id but created within expiry time"""
attendee = AttendeeSubFactory(
created_at=datetime.datetime.utcnow(), modified_at=datetime.datetime.utcnow(),
created_at=datetime.datetime.utcnow(),

Choose a reason for hiding this comment

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

Black would make changes.

@@ -16,7 +16,9 @@ def test_invoice_list_user_error(db, client, jwt, user):
get_invoice(db, user)

response = client.get(
'/v1/event-invoices', content_type='application/vnd.api+json', headers=jwt,
'/v1/event-invoices',

Choose a reason for hiding this comment

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

Black would make changes.

@@ -3,7 +3,7 @@

class SoftDeletionModel(db.Model):
"""
Base model for soft deletion support. All the models which support soft deletion should extend it.
Base model for soft deletion support. All the models which support soft deletion should extend it.

Choose a reason for hiding this comment

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

line too long (102 > 90 characters)

@@ -4,7 +4,7 @@

class SoftDeletionSchema(Schema):
"""
Base Schema for soft deletion support. All the schemas that support soft deletion should extend this schema
Base Schema for soft deletion support. All the schemas that support soft deletion should extend this schema

Choose a reason for hiding this comment

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

line too long (111 > 90 characters)

@@ -19,7 +19,13 @@ def query(self, kwargs):

decorators = (jwt_required,)
schema = ImportJobSchema
data_layer = {'session': db.session, 'model': ImportJob, 'methods': {'query': query,}}
data_layer = {
'session': db.session,

Choose a reason for hiding this comment

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

Black would make changes.

@iamareebjamal iamareebjamal merged commit 2b71304 into development Oct 9, 2020
@iamareebjamal iamareebjamal deleted the reformat branch October 9, 2020 12:10
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