Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOC: document actions #16

Merged
merged 24 commits into from
Nov 4, 2022
Merged

DOC: document actions #16

merged 24 commits into from
Nov 4, 2022

Conversation

jorgepiloto
Copy link
Member

This pull-request documents the different reusable actions available. Information about the inputs and code snippets are provided in the README.rst file to guide users.

@jorgepiloto
Copy link
Member Author

jorgepiloto commented Oct 24, 2022

I thought about creating a Sphinx-based documentation for this. Maybe, this would be interesting as we could test our workflows. Only the release-private and release-public would be untested.

Related with #15.

@github-actions github-actions bot added the maintenance Generic maintenance related label Oct 24, 2022
@jorgepiloto jorgepiloto force-pushed the main branch 5 times, most recently from e01cdaa to 0bace21 Compare October 26, 2022 19:10
@jorgepiloto jorgepiloto force-pushed the doc/actions branch 3 times, most recently from aea729a to fc96d51 Compare October 31, 2022 16:56
@jorgepiloto jorgepiloto marked this pull request as ready for review November 2, 2022 08:34
@jorgepiloto jorgepiloto force-pushed the doc/actions branch 2 times, most recently from 58af8e8 to 89328a1 Compare November 2, 2022 09:27
@jorgepiloto
Copy link
Member Author

Pinging here @PipKat. For some reason, GitHub is not allowing me to add you as a reviewer, Kathy.

@Revathyvenugopal162
Copy link
Contributor

Pinging here @PipKat. For some reason, GitHub is not allowing me to add you as a reviewer, Kathy.

I think you have to add Documentation team in settings/access/collaborators and team and give write access.

@jorgepiloto jorgepiloto requested a review from PipKat November 2, 2022 10:21
@jorgepiloto
Copy link
Member Author

You are right, @Revathyvenugopal162. I just added documentation team to this repository and requested a review by Kathy.

Copy link
Member

@PipKat PipKat left a comment

Choose a reason for hiding this comment

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

Here are my review comments. Lots of nitpicks.

DOC: add Sphinx documentation

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
jorgepiloto and others added 8 commits November 4, 2022 08:48

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Co-authored-by: Revathy Venugopal <104772255+Revathyvenugopal162@users.noreply.github.com>
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
@jorgepiloto jorgepiloto merged commit 40c852b into main Nov 4, 2022
@jorgepiloto jorgepiloto deleted the doc/actions branch November 4, 2022 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Generic maintenance related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants