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

fix classifier to archiveClassifier #14

Merged
merged 1 commit into from
Mar 15, 2024
Merged

Conversation

harukitosa
Copy link
Contributor

@harukitosa harukitosa commented Mar 13, 2024

gradle 7.5.1以降、classifierはdeprecatedになる(以降のversionで消えてbuildが通らなくなるため)、archiveClassifierに変更しました。ご確認お願いします。

https://docs.gradle.org/7.5.1/dsl/org.gradle.api.tasks.bundling.Jar.html

追記 archiveClassifierは5.3から追加されたプロパティ: https://docs.gradle.org/5.3/dsl/org.gradle.api.tasks.bundling.Jar.html

@harukitosa
Copy link
Contributor Author

@wasnot @SojiroNishimura
レビューお願いします!

@wasnot
Copy link
Contributor

wasnot commented Mar 14, 2024

自分は大丈夫と思いますが、時間が許すのなら一応 @nkmrh さんにも確認していただいた方がいいかもしれません。

@harukitosa
Copy link
Contributor Author

@nkmrh
(初めましてで恐縮ですが) レビューお願いします!

Copy link
Collaborator

@nkmrh nkmrh left a comment

Choose a reason for hiding this comment

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

対応ありがとうございます!
余談ですが expo の方ではすでに削除されているようですね。 https://github.com/expo/expo/pull/22537/files

@harukitosa
Copy link
Contributor Author

@nkmrh
ありがとうございます!助かります🙇‍♂️

@harukitosa harukitosa merged commit d014310 into main Mar 15, 2024
1 check passed
@delete-merged-branch delete-merged-branch bot deleted the feature/8368-chat-fix branch March 15, 2024 04:59
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.

4 participants