Skip to content

Fix comments with indents #549

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
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
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.
@greshilov greshilov force-pushed the fix-comments-with-indents branch from 54a8c0a to 093109e Compare January 17, 2021 19:14
@greshilov greshilov force-pushed the fix-comments-with-indents branch from 093109e to 28b2c8e Compare January 17, 2021 19:15
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

Labels

None yet

3 participants

@greshilov @asottile @MasterSlave420