Skip to content

ci: fix opendal-java snapshot releases#2532

Merged
Xuanwo merged 3 commits intomainfrom
fix-java-snapshot-tk2
Jun 25, 2023
Merged

ci: fix opendal-java snapshot releases#2532
Xuanwo merged 3 commits intomainfrom
fix-java-snapshot-tk2

Conversation

@tisonkun
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: tison <wander4096@gmail.com>
@github-actions github-actions Bot added the releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" label Jun 25, 2023
Signed-off-by: tison <wander4096@gmail.com>
Comment thread bindings/java/pom.xml
Comment thread .github/workflows/bindings_java.yml Outdated
Signed-off-by: tison <wander4096@gmail.com>
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks!

@Xuanwo Xuanwo merged commit ad6401c into main Jun 25, 2023
@Xuanwo Xuanwo deleted the fix-java-snapshot-tk2 branch June 25, 2023 13:50
@oowl
Copy link
Copy Markdown
Member

oowl commented Jun 25, 2023

It seems some other PR created by the forked repo can not read github secure key, and then it almost failed https://github.com/apache/incubator-opendal/actions/runs/5370716510/jobs/9742999299?pr=2526

@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Jun 25, 2023

It seems some other PR created by the forked repo can not read github secure key, and then it almost failed https://github.com/apache/incubator-opendal/actions/runs/5370716510/jobs/9742999299?pr=2526

Thanks for report. Fixed in #2534

@PsiACE PsiACE mentioned this pull request Jun 27, 2023
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/ci The PR modifies CI-related content or has a title that begins with "ci"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants