Skip to content
This repository was archived by the owner on Aug 7, 2025. It is now read-only.

CodeArtifact: Add documentation #1775

Merged
merged 16 commits into from
Jun 30, 2025
Merged

CodeArtifact: Add documentation #1775

merged 16 commits into from
Jun 30, 2025

Conversation

skyrpex
Copy link
Contributor

@skyrpex skyrpex commented May 20, 2025

No description provided.

Copy link

github-actions bot commented May 20, 2025

🎊 PR Preview has been successfully built and deployed to https://localstack-docs-preview-pr-1775.surge.sh 🎊

@skyrpex skyrpex marked this pull request as ready for review June 13, 2025 09:00
Copy link
Member

@viren-nadkarni viren-nadkarni left a comment

Choose a reason for hiding this comment

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

Looks great! I've added some suggestions. The PR can be merged in tandem with the 4.6 release.

@skyrpex
Copy link
Contributor Author

skyrpex commented Jun 27, 2025

The coverage for CodeArtifact is wrong data/coverage/codeartifact.json, as it says all methods are implemented but that's not true.

@quetzalliwrites
Copy link
Collaborator

Gracias @skyrpex for your amazing and detailed contribution! ❤️

Your build is failing due to the one line lint rule:
https://github.com/aepfli/markdownlint-rule-max-one-sentence-per-line?tab=readme-ov-file#configuration

Just update your markdown file according to this linting rule, the build will be green again.

@skyrpex skyrpex force-pushed the codeartifact branch 2 times, most recently from 5eddbb9 to 4186c52 Compare June 30, 2025 08:24
@skyrpex skyrpex merged commit 5dcdcb7 into main Jun 30, 2025
5 checks passed
@skyrpex skyrpex deleted the codeartifact branch June 30, 2025 09:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants