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

Typo in Finding and customizing actions #2290

Closed
janiceilene opened this issue Dec 22, 2020 · 5 comments · Fixed by #2326
Closed

Typo in Finding and customizing actions #2290

janiceilene opened this issue Dec 22, 2020 · 5 comments · Fixed by #2326
Labels
content This issue or pull request belongs to the Docs Content team help wanted Anyone is welcome to open a pull request to fix this issue

Comments

@janiceilene
Copy link
Contributor

There is a sentence that has odd wording in the file "main/content/actions/learn-github-actions/finding-and-customizing-actions.md"

Link to the GitHub Help Documents where this can be found:

https://docs.github.com/en/free-pro-team@latest/actions/learn-github-actions/finding-and-customizing-actions#using-release-management-for-your-custom-actions

Screenshot:

image

Suggested:

"[...] that the action will always include the latest updates [...]"

@janiceilene janiceilene added help wanted Anyone is welcome to open a pull request to fix this issue good first issue Good for newcomers content This issue or pull request belongs to the Docs Content team labels Dec 22, 2020
@github-actions github-actions bot added the triage Do not begin working on this issue until triaged by the team label Dec 22, 2020
@tr1ms
Copy link
Contributor

tr1ms commented Dec 23, 2020

I would like to help with this issue and make a PR

@janiceilene
Copy link
Contributor Author

@gsonly That sounds great ✨ You're welcome to open a PR!

@martindes01
Copy link
Contributor

Hi all,

The typo was corrected from "use include" to "use" by commit 09aaf2c, which wasn't linked to this issue.

Although PR #2320 changes this to "include" as suggested, I feel the sentence could be rephrased to better convey its meaning, as well as to follow the style guides more closely.

I might open a pull request if I come up with a nicer way to phrase this.

@janiceilene
Copy link
Contributor Author

Thanks for the clarification @martindes01! I transferred this issue from an internal report and didn't realize it had been fixed already.

@janiceilene janiceilene removed good first issue Good for newcomers triage Do not begin working on this issue until triaged by the team labels Dec 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content This issue or pull request belongs to the Docs Content team help wanted Anyone is welcome to open a pull request to fix this issue
Projects
None yet
4 participants
@janiceilene @martindes01 @tr1ms and others