Skip to content

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
Contributor

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.

3 participants