Skip to content

Semver parsing enhancement#1931

Merged
eyalkoren merged 3 commits intoelastic:masterfrom
eyalkoren:redhat-version-parsing
Jul 28, 2021
Merged

Semver parsing enhancement#1931
eyalkoren merged 3 commits intoelastic:masterfrom
eyalkoren:redhat-version-parsing

Conversation

@eyalkoren
Copy link
Contributor

@eyalkoren eyalkoren commented Jul 22, 2021

What does this PR do?

Closes #1930

A quick and dirty approach, that we can improve later if required.
It looks like using a regex may get quite complicated for this.
If I have more time, I would have made it shorter ( Blaise Pascal)

Checklist

  • This is an enhancement of existing features, or a new feature in existing plugins
    • I have updated CHANGELOG.asciidoc
    • I have added tests that prove my fix is effective or that my feature works
@github-actions github-actions bot added agent-java community Issues and PRs created by the community labels Jul 22, 2021
@ghost
Copy link

ghost commented Jul 22, 2021

💚 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-07-28T05:52:38.650+0000

  • Duration: 53 min 28 sec

  • Commit: 9da30c1

Test stats 🧪

Test Results
Failed 0
Passed 2357
Skipped 19
Total 2376

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 2357
Skipped 19
Total 2376

@tobiasstadler
Copy link
Contributor

Thank You!

@AlexanderWert AlexanderWert removed the community Issues and PRs created by the community label Jul 26, 2021
@eyalkoren eyalkoren requested a review from SylvainJuge July 26, 2021 11:44
Copy link
Member

@SylvainJuge SylvainJuge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for a quick work-around, we can definitely improve that later if needed.

@eyalkoren eyalkoren merged commit c17db61 into elastic:master Jul 28, 2021
@eyalkoren eyalkoren deleted the redhat-version-parsing branch July 28, 2021 07:33
v1v added a commit to v1v/apm-agent-java that referenced this pull request Aug 2, 2021
…junit-support

* upstream/master:
  Updated get-user-teams-membership to version 1.0.3
  ensure that the socket is closed even if there is an exception from the other close (elastic#1946)
  Make sure trace context headers are added only once (elastic#1937)
  Update CHANGELOG.asciidoc
  Ecs reformatting more fields (elastic#1910)
  Use path when high-level framework method is unknown (elastic#1906)
  Semver parsing enhancement (elastic#1931)
v1v added a commit to v1v/apm-agent-java that referenced this pull request Aug 2, 2021
…for-windows-only

* upstream/master: (100 commits)
  [CI] Enable compatibility test matrix for unit tests (elastic#1915)
  Updated get-user-teams-membership to version 1.0.3
  ensure that the socket is closed even if there is an exception from the other close (elastic#1946)
  Make sure trace context headers are added only once (elastic#1937)
  Update CHANGELOG.asciidoc
  Ecs reformatting more fields (elastic#1910)
  Use path when high-level framework method is unknown (elastic#1906)
  Semver parsing enhancement (elastic#1931)
  Bump version.slf4j from 1.7.31 to 1.7.32 (elastic#1933)
  Add description about memory pool metrics to docs (elastic#1925)
  updated team membership check in action
  Add 1.25.0 to cloudfoundry index
  Update CHANGELOG.asciidoc (elastic#1929)
  fixed community label action
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.25.0
  Prepare release 1.25.0 (elastic#1927)
  synchronize json schema specs (elastic#1926)
  added labeling of community issues and PRs
  added community labeler config
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

4 participants