Skip to content
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

[Tool] Configure Dependabot to Ignore ExoPlayer Dependency #19206

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

ParaskP7
Copy link
Contributor

This PR disables Dependabot alerts for the ExoPlayer dependency. Bumping 2.13.3 to the newest version requires lots of changes, including changes to Gutenberg. As such, and as agreed, any update is paused.

For more info see: #17936 (comment)


To test:

No need to explicitly test this change (like done here and here), verifying that no such Dependabot PR will end-up being opened in the future would be enough for now.


Regression Notes

  1. Potential unintended areas of impact

    • N/A
  2. What I did to test those areas of impact (or what existing automated tests I relied on)

    • See To test section above.
  3. What automated tests I added (or what prevented me from doing so)

    • N/A

PR submission checklist:

  • I have completed the Regression Notes.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

UI Changes testing checklist:

  • Portrait and landscape orientations.
  • Light and dark modes.
  • Fonts: Larger, smaller and bold text.
  • High contrast.
  • Talkback.
  • Languages with large words or with letters/accents not frequently used in English.
  • Right-to-left languages. (Even if translation isn’t complete, formatting should still respect the right-to-left layout)
  • Large and small screen sizes. (Tablet and smaller phones)
  • Multi-tasking: Split screen and Pop-up view. (Android 10 or higher)

Bumping '2.13.3' to the newest version requires lots of changes,
including changes to Gutenberg. As such, and as agreed, any update is
paused.

For more info see: https://github.com/wordpress-mobile/
WordPress-Android/pull/17936#issuecomment-1553227875
@wpmobilebot
Copy link
Contributor

WordPress📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress WordPress
FlavorJalapeno
Build TypeDebug
Versionpr19206-12dca79
Commit12dca79
Direct Downloadwordpress-prototype-build-pr19206-12dca79.apk
Note: Google Login is not supported on these builds.

@wpmobilebot
Copy link
Contributor

Jetpack📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack Jetpack
FlavorJalapeno
Build TypeDebug
Versionpr19206-12dca79
Commit12dca79
Direct Downloadjetpack-prototype-build-pr19206-12dca79.apk
Note: Google Login is not supported on these builds.

@ParaskP7
Copy link
Contributor Author

👋 @ravishanker , just a friendly ping here, just in case you missed that, it is a very simple, a 1' min PR at most... 😊 🏓 😊

@ParaskP7 ParaskP7 requested review from thomashorta and removed request for ravishanker October 2, 2023 13:38
@ParaskP7
Copy link
Contributor Author

ParaskP7 commented Oct 2, 2023

👋 @ravishanker I've now unassigned you from this and assigned @thomashorta instead. It seems that you might be busy atm and I don't want to hold this PR any longer. 🙏

Copy link
Contributor

@thomashorta thomashorta left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @ParaskP7 🙇🏼 This looks good to me and it makes sense to disable this dependency update for now.

@thomashorta thomashorta merged commit c2726e3 into trunk Oct 2, 2023
2 checks passed
@thomashorta thomashorta deleted the dependabot/ignore-exoplayer-dependency branch October 2, 2023 14:45
@ParaskP7
Copy link
Contributor Author

ParaskP7 commented Oct 2, 2023

Thank you so much for the blazingly fast review and merge @thomashorta , you are the best! 🙇 ❤️ 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants