Fixed parsing problem with msedgedriver version in release notes #1657
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixed parsing problem with msedgedriver version in release notes generator script and docker image tagging script.
Fixes #1641 where the msedgedriver version was not parsed correctly in release notes and when pushing tags to Docker Hub.
Description
msedgedriver version output looks like this:
When trying to parse the version, we were pulling from the second column. At some point, Microsoft must have changed the location of the version number to the 4th column. Using the 4th column, we get the version:
Motivation and Context
This ensures release notes have the correct version, and that the Docker Hub tags have the correct driver version applied to them.
Types of changes
Checklist