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

cleanup: remove ert.py from cli #5442

Merged
merged 1 commit into from
Nov 17, 2023
Merged

Conversation

LanceEa
Copy link
Contributor

@LanceEa LanceEa commented Nov 17, 2023

Description

This removes the ert.py command which is no longer used and maintained. Allow us to reduce our deps and surface area.

Related Issues

General cleanup

Testing

CI is green, no additional testing needed.

Checklist

  • Does my change need to be backported to a previous release?
  • I made sure to update CHANGELOG.md.
  • This is unlikely to impact how Ambassador performs at scale.
  • My change is adequately tested.
  • I updated DEVELOPING.md with any any special dev tricks I had to use to work on this code efficiently.
  • The changes in this PR have been reviewed for security concerns and adherence to security best practices.

This removes the `ert.py` command which is no
longer used and maintained. Allow us to reduce
our deps and surface area.

Signed-off-by: Lance Austin <[email protected]>
@LanceEa LanceEa marked this pull request as ready for review November 17, 2023 16:34
@LanceEa LanceEa merged commit b2a901d into master Nov 17, 2023
37 checks passed
@LanceEa LanceEa deleted the laustin/remove-grab-snapshots branch November 17, 2023 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants