-
Notifications
You must be signed in to change notification settings - Fork 13
Removes the fixed version number and uses plugin version instead in the unit tests. #254
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
Removes the fixed version number and uses plugin version instead in the unit tests. #254
Conversation
💚 Build Succeeded
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops! Easy mistake to make. Good eye.
Yeah, initially to make other unit tests happy hard coded and focused the version mismatch section. Then forgot to change :) Thanks for quick review. |
@logstashmachine backport 8.x |
(cherry picked from commit c9f0a73)
@logstashmachine backport 8.16 |
(cherry picked from commit c9f0a73)
@logstashmachine backport 8.17 |
(cherry picked from commit c9f0a73)
…) (#256) (cherry picked from commit c9f0a73) Co-authored-by: Mashhur <[email protected]>
…) (#257) (cherry picked from commit c9f0a73) Co-authored-by: Mashhur <[email protected]>
…) (#258) (cherry picked from commit c9f0a73) Co-authored-by: Mashhur <[email protected]>
Removes hard coded version in the unit tests and replaces with the plugin version.
Forgot 🤦 removing it in #247