Skip to content

Conversation

mashhurs
Copy link
Collaborator

@mashhurs mashhurs commented Jan 27, 2025

Removes hard coded version in the unit tests and replaces with the plugin version.
Forgot 🤦 removing it in #247

@mashhurs mashhurs changed the title Removes the fixed version number and uses plugin version instead. Removes the fixed version number and uses plugin version instead in the unit tests. Jan 27, 2025
@mashhurs mashhurs requested a review from donoghuc January 28, 2025 00:01
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

Copy link
Member

@donoghuc donoghuc left a 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.

@mashhurs
Copy link
Collaborator Author

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.

@mashhurs mashhurs merged commit c9f0a73 into elastic:main Jan 28, 2025
2 checks passed
@mashhurs mashhurs deleted the fix-unit-tests-after-version-compatibility-change branch January 28, 2025 00:23
@mashhurs
Copy link
Collaborator Author

@logstashmachine backport 8.x

@mashhurs
Copy link
Collaborator Author

@logstashmachine backport 8.16

@mashhurs
Copy link
Collaborator Author

@logstashmachine backport 8.17

github-actions bot pushed a commit that referenced this pull request Jan 28, 2025
mashhurs added a commit that referenced this pull request Jan 28, 2025
mashhurs added a commit that referenced this pull request Jan 28, 2025
mashhurs added a commit that referenced this pull request Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants