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.

Sign up for GitHub

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

Jump to bottom

Use maven-publish instead of com.vanniktech.maven.publish #75

Merged
merged 12 commits into from
Jul 24, 2024
Merged

Use maven-publish instead of com.vanniktech.maven.publish #75

merged 12 commits into from
Jul 24, 2024

Conversation

jonashendrickx
Copy link
Member

@jonashendrickx jonashendrickx commented Jul 23, 2024 •
edited
Loading

Ticket

n/a

Description

  • Dropping com.vanniktech.maven.publish in favor for the official maven-publish
  • License was modified from Apache License 2.0 to GPL v3.0 to match the repository's license.
  • Fixing workflow linting errors

Shape

n/a

Screenshots

Checklist

I did the following to ensure that my changes were tested thoroughly:

  • A successful deployment was verified. Repository wasn't closed and released but dropped, so it wasn't actually published.

I did the following to ensure that my changes do not introduce security vulnerabilities:

  • __

@jonashendrickx jonashendrickx requested a review from a team as a code owner July 23, 2024 12:47
@jonashendrickx jonashendrickx requested a review from a team as a code owner July 24, 2024 06:34
@jonashendrickx jonashendrickx changed the title Drop vanniktech for publishing the Android library. Use maven-publish Jul 24, 2024
@jonashendrickx jonashendrickx changed the title Use maven-publish Use maven-publish instead of `com.vanniktech.maven.publish Jul 24, 2024
@jonashendrickx jonashendrickx changed the title Use maven-publish instead of `com.vanniktech.maven.publish Use maven-publish instead of com.vanniktech.maven.publish Jul 24, 2024
@jonashendrickx jonashendrickx merged commit 3845d48 into main Jul 24, 2024
5 checks passed
@jonashendrickx jonashendrickx deleted the test-maven-publishing branch July 24, 2024 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@michalchecinski michalchecinski michalchecinski approved these changes

@jrmccannon jrmccannon jrmccannon approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jonashendrickx @jrmccannon @michalchecinski