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

[API-31 enumification] Fix some new enums created in wrong namespaces. #6177

Merged
merged 1 commit into from
Aug 12, 2021

Conversation

jpobst
Copy link
Contributor

@jpobst jpobst commented Aug 11, 2021

Context: #6089

In #6089 we added enumifications for API-31 constants. However some of the namespaces used for these enums did not reflect the changes we apply with metadata to capitalization or plurality. A later "verify any new namespaces added" check revealed these changes.

Example:

Android.App.Appsearch -> Android.App.AppSearch
Android.Location -> Android.Locations

@jpobst jpobst marked this pull request as ready for review August 12, 2021 18:52
@jpobst jpobst requested a review from jonpryor as a code owner August 12, 2021 18:52
@jonpryor
Copy link
Member

jonpryor commented Aug 12, 2021

[API-31 enumification] Fix some enum namespaces names (#6177)

Context: 88438081c6db1e38547ed5afea9aa701329ac01c

We enumified API-31 constants in commit 88438081.  However, some of
the namespaces used for these enums did not reflect the changes we
apply with `metadata` for capitalization or plurality purposes.

A later "verify any new namespaces added" check on @jpobst 's machine
revealed these changes.  (TODO: Export that tool off his machine.)

For example, the following namespaces were renamed:

  * `Android.App.Appsearch` to `Android.App.AppSearch`.
  * `Android.Location` to `Android.Locations`.

@jonpryor jonpryor merged commit cf2a39b into main Aug 12, 2021
@jonpryor jonpryor deleted the api-31-fixups branch August 12, 2021 19:09
jonpryor pushed a commit to jonpryor/xamarin-android that referenced this pull request Aug 25, 2021
Context: 8843808

We enumified API-31 constants in commit 8843808.  However, some of
the namespaces used for these enums did not reflect the changes we
apply with `metadata` for capitalization or plurality purposes.

A later "verify any new namespaces added" check on @jpobst 's machine
revealed these changes.  (TODO: Export that tool off his machine.)

For example, the following namespaces were renamed:

  * `Android.App.Appsearch` to `Android.App.AppSearch`.
  * `Android.Location` to `Android.Locations`.
jonpryor pushed a commit that referenced this pull request Aug 26, 2021
Context: 8843808

We enumified API-31 constants in commit 8843808.  However, some of
the namespaces used for these enums did not reflect the changes we
apply with `metadata` for capitalization or plurality purposes.

A later "verify any new namespaces added" check on @jpobst 's machine
revealed these changes.  (TODO: Export that tool off his machine.)

For example, the following namespaces were renamed:

  * `Android.App.Appsearch` to `Android.App.AppSearch`.
  * `Android.Location` to `Android.Locations`.
@github-actions github-actions bot locked and limited conversation to collaborators Jan 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants