Skip to content

ci: add NeoForge JAR collection to GitHub Release step#49

Merged
turtton merged 1 commit into
mainfrom
ci/multi-loader-release
May 10, 2026
Merged

ci: add NeoForge JAR collection to GitHub Release step#49
turtton merged 1 commit into
mainfrom
ci/multi-loader-release

Conversation

@turtton

@turtton turtton commented May 10, 2026

Copy link
Copy Markdown
Owner

Summary

  • Create GitHub Release ステップの JAR 収集 glob を +fabric-*.jar のみから fabric + neoforge 両対応に拡張
  • sources JAR / client-gametest JAR を除外するフィルタを追加
  • shell: bash を明示(bash 固有構文 shopt, 配列, [[ ]] 使用のため)

Changes

# Before: fabric only
assets=(versions/*/build/libs/connectedtank-"${MOD_VERSION}"+fabric-*.jar)

# After: fabric + neoforge, with exclusion filters
assets=()
for jar in \
  versions/*/build/libs/connectedtank-"${MOD_VERSION}"+fabric-*.jar \
  versions/*/build/libs/connectedtank-"${MOD_VERSION}"+neoforge-*.jar; do
  [[ "$jar" == *-sources.jar ]] && continue
  [[ "$jar" == *-client-gametest.jar ]] && continue
  [[ "$jar" == *-client-gametest-remapped.jar ]] && continue
  assets+=("$jar")
done

Expand JAR glob from fabric-only to fabric+neoforge, exclude sources and test JARs. Add explicit shell: bash for bash-specific syntax.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@turtton turtton merged commit c864075 into main May 10, 2026
2 checks passed
@turtton turtton deleted the ci/multi-loader-release branch May 10, 2026 02:29
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.

1 participant