Skip to content

Improve Readme#6

Merged
abhijoshi2k merged 2 commits into
Readme-Workflows:mainfrom
Andre601:patch-1
May 8, 2021
Merged

Improve Readme#6
abhijoshi2k merged 2 commits into
Readme-Workflows:mainfrom
Andre601:patch-1

Conversation

@Andre601

@Andre601 Andre601 commented May 8, 2021

Copy link
Copy Markdown
Member

Improved the Readme with some better info and examples.

This is currently still a draft since there might be new options available in the near future. (See recent discussion)

Something I wasn't really sure about is if the info about Private repo activities being shown with a custom PAT is true.
If it is true, let me know.

Something else I considered but see an issue with right now is to make an Example Repository in this Organisation you have, that would use this action here. The issue I see is, that Organisations don't have activities, right? In such a case would it need a way to target a specific user for the activities.

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
@PuneetGopinath

PuneetGopinath commented May 8, 2021

Copy link
Copy Markdown
Member

Thanks for the pr. The pr looks good, I can see this is a draft. See the changes I have requested, please resolve them.

@abhijoshi2k abhijoshi2k requested a review from PuneetGopinath May 8, 2021 14:06
@PuneetGopinath PuneetGopinath requested a review from abhijoshi2k May 8, 2021 14:08
@PuneetGopinath PuneetGopinath marked this pull request as ready for review May 8, 2021 14:08
@Andre601 Andre601 marked this pull request as draft May 8, 2021 14:10

@abhijoshi2k abhijoshi2k left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

README looks much better now. Thanks for your efforts @Andre601

@abhijoshi2k abhijoshi2k marked this pull request as ready for review May 8, 2021 14:11
@abhijoshi2k abhijoshi2k merged commit 8c56c8a into Readme-Workflows:main May 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants