Skip to content

update README [show option]#8

Closed
dariocurr wants to merge 1 commit intotest-summary:mainfrom
dariocurr:main
Closed

update README [show option]#8
dariocurr wants to merge 1 commit intotest-summary:mainfrom
dariocurr:main

Conversation

@dariocurr
Copy link
Copy Markdown

@dariocurr dariocurr commented Jun 27, 2022

Hi @ethomson.
I'm Dario, a developer at BuildNN.
Firstly, I want to thank you for the great work you did, I use your action basically everyday and It works really great.

Developing my pytest-summary extension I discovered about the option show that is not mentioned on the README.
Since I consider it helpful, I added it.

PS. any advice on my extension is more than welcome

@dariocurr dariocurr changed the title update README update README [show option] Jun 27, 2022
@dariocurr
Copy link
Copy Markdown
Author

Any update?

@dariocurr
Copy link
Copy Markdown
Author

I guess you don't believe in collaborations.
I opened this PR almost 9 months ago, and today I found out you did it in January by yourself.

A message would be appreciated.

@dariocurr dariocurr closed this May 2, 2023
@ethomson
Copy link
Copy Markdown
Member

ethomson commented May 8, 2023

Hi @dariocurr - apologies that I didn't see this. Indeed, I do believe in collaboration. The README change was included in PR #13.

Thanks for the fix - sorry again that I didn't see the notification for your pull request.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants