-
Notifications
You must be signed in to change notification settings - Fork 150
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
feat(remove-android): Remove mock-adb #6745
Conversation
|
||
- uses: github/codeql-action/autobuild@v2 |
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 github/codeql-action/autobuild@v2
step was added in #5737, and was needed for the C# code in mock-adb. Now that we're back to just JavaScript, we don't need it.
@@ -157,10 +157,8 @@ jobs: | |||
timeout-minutes: 2 | |||
with: | |||
trap-caching: false | |||
languages: javascript |
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.
codeql is expecting to build JavaScript and C#, and complaining when the C# portion is not found. Explicitly telling codeql that we're building JavaScript sets the correct expectation.
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.
Seems okay to me!
Details
Deprecating Android support will touch hundreds of files, so we're breaking it up into more manageable units. This PR removes the files to build and configure mock-adb. Later PR's will remove the .NET build tools and the code that was added to enable e2e tests to force the code to start the session with an empty setup store.
Motivation
Android deprecation feature
Context
Pull request checklist
yarn fastpass
yarn test
)<rootDir>/test-results/unit/coverage
fix:
,chore:
,feat(feature-name):
,refactor:
). SeeCONTRIBUTING.md
.