Skip to content

[Demo] Slash command code formatting - Python#11

Closed
retepsnave wants to merge 10 commits into
peter-evans:masterfrom
retepsnave:patch-3
Closed

[Demo] Slash command code formatting - Python#11
retepsnave wants to merge 10 commits into
peter-evans:masterfrom
retepsnave:patch-3

Conversation

@retepsnave

Copy link
Copy Markdown

Slash command code formatting - Python

Slash commands can be used to execute code formatting tools on pull request branches.
The slash commands are dispatched to slash-command-dispatch-processor repository to be processed by workflows here.

The code formatting workflows do the following:

  1. Check out the pull request branch
  2. Execute the code formatting tool
  3. Commit the changes to the pull request branch
  4. Add a 🎉 reaction to the comment containing the slash command

The following command formats Python using autopep8

/autopep8

The following command formats Python using Black

/black

The demo can be reset with this command.

/reset-demo

Feel free to try it out! 💬

@peter-evans peter-evans added the demo This issue or pull request is for demonstration purposes label Dec 4, 2019
@peter-evans

Copy link
Copy Markdown
Owner

/autopep8

@peter-evans

Copy link
Copy Markdown
Owner

/reset-demo

@peter-evans

Copy link
Copy Markdown
Owner

/black

@peter-evans

Copy link
Copy Markdown
Owner

/reset-demo

@chiefjester

Copy link
Copy Markdown

/black

@chiefjester

Copy link
Copy Markdown

/autopep8

@peter-evans

Copy link
Copy Markdown
Owner

/reset-demo

@stack72

stack72 commented Feb 25, 2020

Copy link
Copy Markdown

/black

@peter-evans

Copy link
Copy Markdown
Owner

/reset-demo

1 similar comment
@peter-evans

Copy link
Copy Markdown
Owner

/reset-demo

@peter-evans

Copy link
Copy Markdown
Owner

/reset-demo

@peter-evans

Copy link
Copy Markdown
Owner

@stack72 Sorry it didn't work when you tried it. I've just realised that the demo gets stuck if /reset-demo isn't used after every /black or /autopep8. I need to find a better way to reset the demo!

For now I've recreated the PR here: #28

@denizs

denizs commented Sep 17, 2020

Copy link
Copy Markdown

/black

@ckolyakovmmc

Copy link
Copy Markdown

/reset-demo

@nhuray

nhuray commented Feb 27, 2021

Copy link
Copy Markdown

/awesome

@mtaha3249

Copy link
Copy Markdown

/reset-demo

@mtaha3249

Copy link
Copy Markdown

/black

@cdnninja

Copy link
Copy Markdown

/reset-demo

@chubchubsancho

Copy link
Copy Markdown

/autopep8

@s2mr

s2mr commented Jan 18, 2022

Copy link
Copy Markdown

/black

@ramosian-glider

Copy link
Copy Markdown

/autopep8

@ramosian-glider

Copy link
Copy Markdown

/reset-demo

@niallscc

Copy link
Copy Markdown

/reset-demo

1 similar comment
@ppigazzini

Copy link
Copy Markdown

/reset-demo

@ppigazzini

Copy link
Copy Markdown

/black

1 similar comment
@frap

frap commented Nov 30, 2022

Copy link
Copy Markdown

/black

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

Labels

demo This issue or pull request is for demonstration purposes

Projects

None yet

Development

Successfully merging this pull request may close these issues.