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.

Sign up for GitHub

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

Jump to bottom

[MS-755] feat: Add aws local developemnt tools #10

Merged
merged 1 commit into from
Oct 4, 2024
Merged

[MS-755] feat: Add aws local developemnt tools #10

merged 1 commit into from
Oct 4, 2024

Conversation

piotrgrundas
Copy link
Collaborator

@piotrgrundas piotrgrundas commented Oct 4, 2024 •
edited by jira bot
Loading

Resolves:

@piotrgrundas piotrgrundas force-pushed the MS-755/localstack branch 4 times, most recently from f588977 to 4d2d2a1 Compare October 4, 2024 14:11
piotrgrundas
piotrgrundas commented Oct 4, 2024
View reviewed changes
docker-compose.yml
image: localstack/localstack:latest
environment:
# LocalStack configuration: https://docs.localstack.cloud/references/configuration/
- AWS_DEFAULT_REGION=${AWS_REGION?:AWS_REGION env is required}
Copy link
Collaborator Author

@piotrgrundas piotrgrundas Oct 4, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

Intended - localstack ingores AWS_REGION variable and uses his default one value

@piotrgrundas piotrgrundas merged commit 4cf561c into develop Oct 4, 2024
1 check passed
@piotrgrundas piotrgrundas deleted the MS-755/localstack branch October 4, 2024 17:26
piotrgrundas added a commit that referenced this pull request Oct 24, 2024
@piotrgrundas
[MS-755] feat: Add aws local developemnt tools (#10)
8ac0744
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JanChodorowski JanChodorowski JanChodorowski approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@piotrgrundas @JanChodorowski