Devs: please run your tests before asking to merge a PR

23 views
Skip to first unread message

Edward K. Ream

unread,
Mar 21, 2022, 11:06:31 AM3/21/22
to leo-editor
I'm trusting that all devs run Leo's test-all command and run pylint on any changed files as part of their testing for a PR.

I'm going to know if you haven't done this: after merging a PR I always run test-all and run pylint on all files.

Please don't make extra work for me.

Edward

Félix

unread,
Mar 21, 2022, 7:37:56 PM3/21/22
to leo-editor

Oops - my bad!

(I knew i had forgotten something important last night when i was wrapping things up for the p.r.)

Sorry again for that: I wont forget next time

Félix

Edward K. Ream

unread,
Mar 21, 2022, 7:44:39 PM3/21/22
to leo-editor
On Mon, Mar 21, 2022 at 6:37 PM Félix <felix...@gmail.com> wrote:

Oops - my bad!

(I knew i had forgotten something important last night when i was wrapping things up for the p.r.)

Sorry again for that: I wont forget next time

No problem.  You weren't the only one :-)

Edward
Reply all
Reply to author
Forward
0 new messages