Skip to content

refactor(e2e-tests): names and logger #6999

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
MiguelAAe merged 8 commits into from
Dec 10, 2025
Merged

refactor(e2e-tests): names and logger #6999

MiguelAAe merged 8 commits into from
Dec 10, 2025

Conversation

@MiguelAAe
Copy link
Contributor

@MiguelAAe MiguelAAe commented Dec 10, 2025 •
edited
Loading

Why

Issues

Motivation

As the code base is growing its important to keep the code tidy

What

Summary of changes

  • tidy up names
  • change logger and added it to more dependencies
  • change way of deleting resources with k8s client deleteAllOf func

Checklist

  • Added/updated unit tests
  • Added/updated documentation
  • Checked for typos in variable names, comments, etc.
  • Added licences for new files

Testing

@MiguelAAe MiguelAAe requested a review from lc525 as a code owner December 10, 2025 16:40
@MiguelAAe MiguelAAe merged commit bfa38fb into SeldonIO:v2 Dec 10, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@domsolutions domsolutions domsolutions approved these changes

@lc525 lc525 Awaiting requested review from lc525 lc525 is a code owner

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

@MiguelAAe @domsolutions