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

v2.0.7: Support JitPack repository #48

Merged
merged 15 commits into from
Oct 19, 2021
Merged

v2.0.7: Support JitPack repository #48

merged 15 commits into from
Oct 19, 2021

Conversation

irgaly
Copy link
Contributor

@irgaly irgaly commented Aug 31, 2021

related: #47

@tomoima525

こちらのマージ後に 2.0.7 release tag を作成していただくと、JitPack から com.github.tomoima525.debot:debot:2.0.7 で取得できるようになります。

JitPack のビルド状況は https://jitpack.io/#tomoima525/debot でご確認いただけます。


jcenter へのアップロードをやめて、JitPack を利用するようにします。
方針としては、CI / ビルド環境 (Gradle / compileSdk =30) は最新となるようにしつつ、ライブラリの成果物は 2.0.6 から変化がないようにします。

  • bump version 2.0.7
  • use JDK 11
  • use Gradle 7.2
  • use AGP 7.0.1
  • compileSdk = 30
  • jcenter() 依存削除
  • bintray publisher 削除
  • Jetifier 削除: v2.0.6 から AndroidX に対応済みのため
  • JitPack 対応
    • Maven Publish Plugin 設定の修正
    • GroupID = JitPack の com.github.tomoima525.debot へ変更
  • CircleCI テスト修正
  • CircleCI bintray へのアップロードを削除
  • README.md の更新

@irgaly irgaly changed the title v2.0.7: Support Jitpack repository v2.0.7: Support JitPack repository Aug 31, 2021
@tomoima525 tomoima525 self-requested a review August 31, 2021 22:30
@irgaly
Copy link
Contributor Author

irgaly commented Sep 30, 2021

@tomoima525
こちらご対応よろしくお願いします。
レビューしていただいて問題がなければ、その後やることとしては、以下の2点だけです。

  • マージする
  • 2.0.7 tag を push する

@tomoima525
Copy link
Owner

@irgaly 返答おくれてすいません。LGTM!

@tomoima525 tomoima525 merged commit 10d2736 into tomoima525:master Oct 19, 2021
@tomoima525
Copy link
Owner

@irgaly Sample のビルドが fail していたので #49 で対応しました。 2.0.7が publishされたの確認しました。対応いただきありがとうございます!!!

@irgaly
Copy link
Contributor Author

irgaly commented Oct 22, 2021

ありがとうございます!

@irgaly irgaly deleted the jitpack branch October 22, 2021 06:49
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.

2 participants