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

Conversation

@anisaoshafi
Copy link
Contributor

@anisaoshafi anisaoshafi commented Jan 2, 2025

Added small clarification on configuration of profiles.
In my onboarding experience, the ./localstack/ directory was not created when I was trying profiles out and I spent some minutes getting lost on whether this directory was supposed to get automatically created or not. Which should be if you install localstack via cli.

@github-actions
Copy link

github-actions bot commented Jan 2, 2025

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

@anisaoshafi
Copy link
Contributor Author

Seems the link checks are failing after today's upgrade of DavidAnson/markdownlint-cli2-action@v18 --> DavidAnson/markdownlint-cli2-action@v19 f319786 👀

@bentsku
Copy link
Contributor

bentsku commented Jan 3, 2025

\cc @HarshCasper any idea why the linting action now fails? Should we revert back to 18?

Copy link
Contributor

@bentsku bentsku left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for adding the clarification! I guess we can merge even with the lint failure.

@anisaoshafi anisaoshafi merged commit db25b3e into main Jan 7, 2025
3 of 5 checks passed
@anisaoshafi anisaoshafi deleted the nit-config-profiles branch January 7, 2025 10:14
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