-
Notifications
You must be signed in to change notification settings - Fork 850
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
Implement space_after_not option #6050
base: master
Are you sure you want to change the base?
Conversation
607af39
to
f66a86a
Compare
Thanks for the PR. Going to mark this one as on hold for now. There was some discussion back in #5308, where we were unsure if we should add this option. Again, I want to thank you for stepping up to work on this. We'll follow up once we've had a chance to discuss this one. |
@ytmimi Sure. Is there something I can help with in the meantime? |
@x17jiri not at this time. This is something that the team needs to discuss to unblock. I brought this up for discussion in our last team meeting, but it hasn't been addressed yet because the team couldn't meet at the scheduled time. The discussion might continue on the linked thread, but if it doesn't I'll do my best to bring this one up again during our next team meeting |
@x17jiri my last comment was referring to this one being blocked until the team has a chance to discuss this, but I'm now realizing that maybe you were asking about helping out with other issues. If that's the case, then I'm sure I could point you in the right direction! |
Yes, that's what I meant :-) |
Fantastic! I'll dig through the backlog when I've got a little more free time and tag you in one or two issues that I think would be a good fit for a new contributor. |
There has been request for this some time ago #5308
I would also like to have this option