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

WRITEME: Link to snippets anywhere #5392

Merged
merged 1 commit into from
Sep 20, 2023

Conversation

Laren-AWS
Copy link
Contributor

Search for snippets across the entire codebase instead of only in a single subfolder.
This lets us generate READMEs in a language/service subfolder that link to action snippets that are implemented elsewhere, such as a workflow scenario or cross-service example.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@Laren-AWS Laren-AWS self-assigned this Sep 15, 2023
@Laren-AWS Laren-AWS added On Call Review needed This work needs an on-call review Text Review not needed This contains no text which needs review Task labels Sep 15, 2023
@DavidSouther DavidSouther removed the On Call Review needed This work needs an on-call review label Sep 19, 2023
Copy link
Contributor

@DavidSouther DavidSouther left a comment

Choose a reason for hiding this comment

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

I think there's a lot of refactoring left to do, but pulling out Snippet is a good start.

@DavidSouther DavidSouther added the On Call Review complete On call review complete label Sep 20, 2023
@beqqrry-aws beqqrry-aws merged commit a3b38cc into awsdocs:main Sep 20, 2023
8 checks passed
@Laren-AWS Laren-AWS deleted the writeme-snippets-anywhere branch June 10, 2024 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
On Call Review complete On call review complete Text Review not needed This contains no text which needs review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants