[yt-dev] Re: slash commands

Madicken merged the PR, so now these commands should work; it needed a bit of adjustment, but we now have: /help /format /rebase /isort /black You can see some of our attempts and experiments here: https://github.com/yt-project/yt/pull/2718 On Wed, Jul 29, 2020 at 11:21 AM Matthew Turk wrote:

I have transferred the repository! Hooray!

On Wed, Jul 29, 2020 at 10:13 AM Corentin Cadiou wrote:

...

That sounds like a really good thing to have. It will probably lower the contribution barrier to what it was before YTEP0037, with the extra benefit of a cleaner code!

+1

Corentin

29 Jul 2020 15:57:13 Britton Smith :

I think this sounds like a great idea. I like the idea of not having to ask contributors to do a lot of PR reformatting on their own. +1 to all of it.

Britton

On Mon, Jul 27, 2020 at 10:15 PM Matthew Turk wrote:

...

I got a couple questions off-list about what this means -- basically, with YTEP 37 we've added some restrictions to what we accept, particularly in how we format code, etc. This can be tricky and hard to address for contributors, and so what these commands do is implement a way to automate that. For instance, I've added a command /black so that if you leave a comment on a pull request that is just "/black" it will run black on the code, and then update the pull request. Same for /isort , which runs isort. This should lower the barrier about some of our stylistic bits that could otherwise be gatekeep-y.

On Mon, Jul 27, 2020 at 3:04 PM Matthew Turk wrote:

...

Hi folks,

I've issued a PR that adds a few chatops-style commands to our pull requests.

https://github.com/yt-project/yt/pull/2789

There are links in there to the issue:

https://github.com/yt-project/yt/issues/2776

which itself has a few links. Please feel free to chime in there, but I think this could make our development workflow a lot nicer -- we could stop asking people to rebase, for instance, and instead just have it done almost-automatically by a bot. And same for black, isort, etc. I'll also add a help command, now that I'm thinking of it...

Please leave feedback -- if I get some buy-in from folks I'll transfer the slash-command-processor repo to the yt-project org and then we can move forward with it a bit.

-Matt

_______________________________________________ yt-dev mailing list -- yt-dev@python.org To unsubscribe send an email to yt-dev-leave@python.org https://mail.python.org/mailman3/lists/yt-dev.python.org/ Member address: brittonsmith@gmail.com

_______________________________________________ yt-dev mailing list -- yt-dev@python.org To unsubscribe send an email to yt-dev-leave@python.org https://mail.python.org/mailman3/lists/yt-dev.python.org/ Member address: matthewturk@gmail.com