Skip to content

Backport selenium fix dotnet/aspnetcore#30576 to 5.0 #30621

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

Merged
merged 1 commit into from
Mar 5, 2021

Conversation

TanayParikh
Copy link
Contributor

Original PR: #30576

* Update selenium version

* Update more versions

* Update Versions.props

* Update selenium-config.json

* Update Versions.props
@TanayParikh TanayParikh requested a review from dougbu as a code owner March 3, 2021 22:23
@TanayParikh TanayParikh requested a review from JunTaoLuo March 3, 2021 22:23
@TanayParikh
Copy link
Contributor Author

@JunTaoLuo / @dougbu can this be merged?

@Pilchie Pilchie added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Mar 4, 2021
@dougbu
Copy link
Contributor

dougbu commented Mar 4, 2021

can this be merged?

Definitely❕

@TanayParikh TanayParikh merged commit 5987589 into release/5.0 Mar 5, 2021
@TanayParikh TanayParikh deleted the taparik/applySeleniumUpdatesOn5-0 branch March 5, 2021 00:10
@dougbu
Copy link
Contributor

dougbu commented Mar 5, 2021

My bad. Should have reminded you to wait 'til branch was open for this. @wtgodbe will this 'release/5.0' change cause CQB issues❔

@wtgodbe
Copy link
Member

wtgodbe commented Mar 5, 2021

Thanks for catching it quickly, just need to cancel the build to avoid any fallout (which I just did: https://dev.azure.com/dnceng/internal/_build/results?buildId=1023916&view=results)

@ghost
Copy link

ghost commented Mar 5, 2021

Hi @wtgodbe. It looks like you just commented on a closed PR. The team will most probably miss it. If you'd like to bring something important up to their attention, consider filing a new issue and add enough details to build context.

@dougbu dougbu added this to the 5.0.5 milestone Jul 6, 2021
@dougbu dougbu added the tell-mode Indicates a PR which is being merged during tell-mode label Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework tell-mode Indicates a PR which is being merged during tell-mode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants