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
add statement api #38
Conversation
9721050
to
560584f
Compare
.github/workflows/main.yml
Outdated
run: mvn install -Dgpg.skip=true | ||
- name: Test | ||
run: mvn test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
testされてなかったのでする
|
||
* Init source. | ||
|
||
Please see https://github.com/payjp/payjp-java/releases |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Github見てもらう
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
16,17,21のサポートをしたいがmockitoの大幅ver up(とそれに伴うリファクタ)が必要
|
||
* Init source. | ||
|
||
Please see https://github.com/payjp/payjp-java/releases |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Github見てもらう
RequestExample.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/docs/api
に集約するため削除
@@ -17,12 +17,6 @@ | |||
<artifactId>jaxb-api</artifactId> | |||
<version>2.3.0</version> | |||
</dependency> | |||
<dependency> | |||
<groupId>com.google.guava</groupId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vulnerabilitiesアラートが出ている。動いてないテストにのみ使用のためテストごと削除。
https://mvnrepository.com/artifact/jp.pay/payjp-java/0.6.1
<source>1.6</source> | ||
<target>1.6</target> | ||
<source>1.8</source> | ||
<target>1.8</target> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jdk >= 16に向けてあげておく
@@ -44,7 +44,4 @@ public void setDeleted(Boolean deleted) { | |||
public Boolean getLivemode() { | |||
return livemode; | |||
} | |||
public void setLivemode(Boolean livemode) { | |||
this.livemode = livemode; | |||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
参照のみの場所にsetがある、があちこちあるので目に見えたとこだけ削除
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
参照のみの場所にsetがある、があちこちあるので目に見えたとこだけ削除
@@ -65,23 +65,6 @@ public static <T> void verifyGet( | |||
APIResource.RequestType.NORMAL, RequestOptions.getDefault()); | |||
} | |||
|
|||
public static <T> void verifyGet( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
使っていない・予定もないので削除
@@ -97,23 +80,6 @@ public static <T> void verifyPost( | |||
APIResource.RequestType.NORMAL, RequestOptions.getDefault()); | |||
} | |||
|
|||
public static <T> void verifyPost( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
同上
package jp.pay; | ||
|
||
import com.google.common.base.Joiner; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
アラート出てるguava消すため(大した使い方してない)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
guava消すため。forループの中で使ってるけどforループが回ってない(=ファイル全体で何もしてない)ので削除。
560584f
to
0e97ab8
Compare
0e97ab8
to
d2a7371
Compare
No description provided.