Skip to content

fix(ci): refactor workflows to fix failing CI#20

Open
druid-infra wants to merge 1 commit intomasterfrom
fix/ci-refactor-20260324
Open

fix(ci): refactor workflows to fix failing CI#20
druid-infra wants to merge 1 commit intomasterfrom
fix/ci-refactor-20260324

Conversation

@druid-infra
Copy link
Collaborator

Changes

This PR fixes the failing CI workflows in the scrolls repository:

Issues Fixed

  1. Node.js 20 deprecation warnings - Updated action versions
  2. Prebuild LGSM failing - Added better error handling, timeouts, and fail-fast: false
  3. Fetch Spigot failing - Added network timeouts and retries
  4. Fetch Forge failing - Fixed job name (was copy-pasted as 'build-spigot'), added error handling

Key Changes

  • actions/checkout@v3v4
  • actions/setup-java@v3v4
  • Added FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: true env var
  • Added timeout-minutes to all jobs (prevents hanging)
  • Changed fail-fast: false for matrix jobs (one failure doesn't cancel all)
  • Added wget timeouts and retries for network resilience
  • Added cleanup step for failed prebuild jobs
  • Fixed copy-paste error in fetch-forge.yml job name

Testing

  • Prebuild LGSM should complete without hanging
  • Fetch Spigot should handle network issues gracefully
  • Fetch Forge should properly parse download links

Ready for review! 🚀

- Update actions/checkout@v3 -> v4 to fix Node.js 20 deprecation
- Update actions/setup-java@v3 -> v4
- Add FORCE_JAVASCRIPT_ACTIONS_TO_NODE24 env var
- Add timeout-minutes to prevent hanging jobs
- Change fail-fast: false for matrix jobs (don't cancel all on one failure)
- Fix job name in fetch-forge (was copy-pasted as 'build-spigot')
- Add better error handling and logging
- Add cleanup step for failed prebuild jobs
- Add wget timeouts and retries for network resilience
@druid-infra
Copy link
Collaborator Author

Error: This repo is not allowlisted for Atlantis.

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