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.

Sign up for GitHub

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

Jump to bottom

Fix comments with indents #549

Merged
merged 1 commit into from
Jan 17, 2021
Merged

Fix comments with indents #549

merged 1 commit into from
Jan 17, 2021

Conversation

greshilov
Copy link
Contributor

Fix for #548.

asottile
asottile reviewed Jan 17, 2021
View reviewed changes
requirements-dev.txt Outdated
@@ -1,3 +1,4 @@
covdefaults
coverage
pytest
ruamel.yaml
Copy link
Member

Choose a reason for hiding this comment

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

this is not correct

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Deleted.

I was unable to run tests without this package.

Copy link
Member

Choose a reason for hiding this comment

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

you probably have extremely old setuptools / pip / tox / virtualenv-- upgrade those

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, looks like I misconfigured something.

asottile
asottile approved these changes Jan 17, 2021
View reviewed changes
Copy link
Member

@asottile asottile left a comment

Choose a reason for hiding this comment

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

@asottile asottile merged commit 43f5ffa into pre-commit:master Jan 17, 2021
MasterSlave420
MasterSlave420 reviewed Feb 2, 2021
View reviewed changes
Copy link

@MasterSlave420 MasterSlave420 left a comment

Choose a reason for hiding this comment

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

Thanks

@ghost
Copy link

ghost commented May 14, 2021

Thanks for adding this.

@asottile do you know when the next release would be approximately?
There's a few additions I'd like to use, this included 😄

@asottile
Copy link
Member

Thanks for adding this.

@asottile do you know when the next release would be approximately?
There's a few additions I'd like to use, this included 😄

the next release will be when the next release is

please don't ask for releases (from anyone, not just me), it's borderline rude -- if you want free work consider sponsoring

@ghost
Copy link

ghost commented May 14, 2021

@asottile ,
Sorry, about that.
I’ve better informed myself and I now see that was very bad etiquette on my behalf. Thank you for correcting it :)


I appreciate you and your work.





Please accept my apology. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MasterSlave420 MasterSlave420 MasterSlave420 left review comments

@asottile asottile asottile approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@greshilov @asottile @MasterSlave420