Thanks to visit codestin.com
Credit goes to github.com

Skip to content

remove deprecated "infra" CLI command #9573

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
Merged
merged 1 commit into from
Nov 7, 2023
Merged

remove deprecated "infra" CLI command #9573

merged 1 commit into from
Nov 7, 2023

Conversation

alexrashed
Copy link
Member

Motivation

Removes the only deprecated CLI command in localstack-core: "infra"

Changes

  • Removes the infra CLI command group

@alexrashed alexrashed added the semver: major Breaking changes which can be included in a major release only label Nov 7, 2023
@alexrashed alexrashed added this to the 3.0 milestone Nov 7, 2023
@alexrashed alexrashed self-assigned this Nov 7, 2023
@alexrashed alexrashed requested a review from thrau as a code owner November 7, 2023 13:05
@alexrashed alexrashed mentioned this pull request Nov 7, 2023
Merged
20 tasks
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 7, 2023

LocalStack Community integration with Pro

       2 files  ±0         2 suites  ±0   1h 8m 15s ⏱️ + 8m 48s
2 302 tests ±0  1 785 ✔️ ±0  517 💤 ±0  0 ❌ ±0 
2 303 runs  ±0  1 785 ✔️ ±0  518 💤 ±0  0 ❌ ±0 

Results for commit 7c3f508. ± Comparison against base commit dffe61f.

@alexrashed alexrashed requested a review from dfangl November 7, 2023 14:39
@coveralls
Copy link

Coverage Status

coverage: 84.236% (+0.01%) from 84.224%
when pulling 7c3f508 on remove-deprecated-cli-commands
into dffe61f on release/v3.0.

dfangl
dfangl approved these changes Nov 7, 2023
View reviewed changes
Copy link
Member

@dfangl dfangl left a comment •
edited
Loading

Choose a reason for hiding this comment

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

🟥🟥🟥🟥🟥

@alexrashed alexrashed merged commit 0b66e9d into release/v3.0 Nov 7, 2023
@alexrashed alexrashed deleted the remove-deprecated-cli-commands branch November 7, 2023 16:57
alexrashed added a commit that referenced this pull request Nov 7, 2023
@alexrashed
remove deprecated "infra" CLI command (#9573)
65b4df6
localstack-bot pushed a commit that referenced this pull request Nov 9, 2023
@alexrashed
remove deprecated "infra" CLI command (#9573)
0e90639
alexrashed added a commit that referenced this pull request Nov 9, 2023
@alexrashed
remove deprecated "infra" CLI command (#9573)
7846393
alexrashed added a commit that referenced this pull request Nov 9, 2023
@alexrashed
remove deprecated "infra" CLI command (#9573)
3fc7878
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dfangl dfangl dfangl approved these changes

@dominikschubert dominikschubert Awaiting requested review from dominikschubert

@thrau thrau Awaiting requested review from thrau

Assignees

@alexrashed alexrashed

Labels
semver: major Breaking changes which can be included in a major release only
Projects
None yet
Milestone
3.0
Development

Successfully merging this pull request may close these issues.
3 participants
@alexrashed @coveralls @dfangl