-
Notifications
You must be signed in to change notification settings - Fork 533
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
[Xamarin.Android.Build.Tests] Fix ActionBarSherlock URL #8926
[Xamarin.Android.Build.Tests] Fix ActionBarSherlock URL #8926
Conversation
On 2024-May-01, xamarin/monodroid-samples was updated so that the `main` branch no longer contained samples for Classic Xamarin.Android. This in turn meant the removal of the ActionBarSherlock sample. This change in turn broke xamarin-android unit tests, which attempt to download this URL, which no longer exists: * <https://github.com/xamarin/monodroid-samples/blob/main/ActionBarSherlock/ActionBarSherlock/Jars/ActionBarSherlock-4.3.1.zip?raw=true> Oops. As part of the removal of Classic Xamarin.Android samples, a new `monodroid-samples/archived-xamarin` branch was created, which is the state of the `main` branch before all old samples were removed. Update the `BindingBuildTest.RemoveEventHandlerResolution()` integration test to use the correct URL for ActionBarSherlock.
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.
The new URL works for me in a browser, so should fix the test.
The new test is green, going to merge to see if our various release branches can be greener: There are other failures, I think were caused by #8921. I think if a PR is merged between build & test lanes it can cause failures on PRs. |
On 2024-May-01, xamarin/monodroid-samples was updated so that the `main` branch no longer contained samples for Classic Xamarin.Android. This in turn meant the removal of the ActionBarSherlock sample. This change in turn broke xamarin-android unit tests, which attempt to download this URL, which no longer exists: * <https://github.com/xamarin/monodroid-samples/blob/main/ActionBarSherlock/ActionBarSherlock/Jars/ActionBarSherlock-4.3.1.zip?raw=true> Oops. As part of the removal of Classic Xamarin.Android samples, a new `monodroid-samples/archived-xamarin` branch was created, which is the state of the `main` branch before all old samples were removed. Update the `BindingBuildTest.RemoveEventHandlerResolution()` integration test to use the correct URL for ActionBarSherlock.
On 2024-May-01, xamarin/monodroid-samples was updated so that the `main` branch no longer contained samples for Classic Xamarin.Android. This in turn meant the removal of the ActionBarSherlock sample. This change in turn broke xamarin-android unit tests, which attempt to download this URL, which no longer exists: * <https://github.com/xamarin/monodroid-samples/blob/main/ActionBarSherlock/ActionBarSherlock/Jars/ActionBarSherlock-4.3.1.zip?raw=true> Oops. As part of the removal of Classic Xamarin.Android samples, a new `monodroid-samples/archived-xamarin` branch was created, which is the state of the `main` branch before all old samples were removed. Update the `BindingBuildTest.RemoveEventHandlerResolution()` integration test to use the correct URL for ActionBarSherlock.
* main: Bump to xamarin/Java.Interop/main@78d5937 (#8935) [docs] Add "Getting Started" docs (#8934) [Xamarin.Android.Build.Tests] Fix ActionBarSherlock URL (#8926) Update README (#8913) Bumps to xamarin/Java.Interop/main@4e893bf (#8924) Bump to dotnet/installer@fa261b952d 9.0.100-preview.5.24253.16 (#8921) [Mono.Android] Dispose of the `RunnableImplementor` on error (#8907) Bump NDK to r26d (#8868)
On 2024-May-01, xamarin/monodroid-samples was updated so that the
main
branch no longer contained samples for Classic Xamarin.Android. This in turn meant the removal of the ActionBarSherlock sample.This change in turn broke xamarin-android unit tests, which attempt to download this URL, which no longer exists:
Oops.
As part of the removal of Classic Xamarin.Android samples, a new
monodroid-samples/archived-xamarin
branch was created, which is the state of themain
branch before all old samples were removed.Update the
BindingBuildTest.RemoveEventHandlerResolution()
integration test to use the correct URL for ActionBarSherlock.