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

Make readme.md structure more consistent with C# SDK #73

Merged
merged 2 commits into from
May 1, 2024

Conversation

sunnywu
Copy link
Contributor

@sunnywu sunnywu commented May 1, 2024

@sunnywu sunnywu requested review from IanDBird and jon8787 May 1, 2024 01:09
@sunnywu sunnywu self-assigned this May 1, 2024
README.md Outdated
UID2 is released under the Apache license. [See LICENSE](https://github.com/IABTechLab/uid2-android-sdk/blob/main/LICENSE.md) for details.
The UID2 SDK is a standalone headless library defined and published via Maven Central. As such the `dev-app` is the primary way for developing the SDK. Use Android Studio to open the root folder to begin development.

ode style is enforced via the [Spotless Gradle plugin](https://github.com/diffplug/spotless) (using [ktlint](https://pinterest.github.io/ktlint/)). You can fix any formatting issues by running `./gradlew spotlessApply`.
Copy link

Choose a reason for hiding this comment

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

typo? was already there before

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@IanDBird IanDBird force-pushed the syw-UID2-2947-ios-sdk-readme branch from a5e2ace to 2532b19 Compare May 1, 2024 09:23
Copy link
Collaborator

@IanDBird IanDBird left a comment

Choose a reason for hiding this comment

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

LGTM

@IanDBird IanDBird merged commit 4d53295 into main May 1, 2024
2 checks passed
@IanDBird IanDBird deleted the syw-UID2-2947-ios-sdk-readme branch May 1, 2024 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants