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

Remove helper class manager #2100

Merged
merged 2 commits into from
Sep 1, 2021
Merged

Remove helper class manager #2100

merged 2 commits into from
Sep 1, 2021

Conversation

felixbarny
Copy link
Member

@felixbarny felixbarny commented Sep 1, 2021 •
edited by zube bot
Loading

What does this PR do?

Checklist

Relates to #1337

@apmmachine
Copy link
Contributor

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-09-01T12:01:43.167+0000

  • Duration: 55 min 30 sec

  • Commit: 9aeeef6

Test stats 🧪

Test Results
Failed 0
Passed 2390
Skipped 19
Total 2409

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 2390
Skipped 19
Total 2409

@felixbarny felixbarny merged commit 7752081 into elastic:master Sep 1, 2021
@felixbarny
Copy link
Member Author

Merging without review as this is just deleting unused classes

@felixbarny felixbarny deleted the remove-helper-class-manager branch September 1, 2021 12:58
@eyalkoren
Copy link
Contributor

My beautiful baby 😢

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SylvainJuge SylvainJuge Awaiting requested review from SylvainJuge

Assignees
No one assigned
Labels
agent-java
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@felixbarny @apmmachine @eyalkoren