Skip to content

Remove API Challenges modules#65

Merged
eviltester merged 1 commit into
masterfrom
codex/split-api-challenges-cleanup
Jul 16, 2026
Merged

Remove API Challenges modules#65
eviltester merged 1 commit into
masterfrom
codex/split-api-challenges-cleanup

Conversation

@eviltester

Copy link
Copy Markdown
Owner

Summary

This completes the API Challenges repo split from the Thingifier side after creating the standalone eviltester/apichallenges repository.

  • removes challenger and challengerAuto from the Maven reactor
  • deletes the old API Challenges source, tests, content, and Docker files from this repo
  • updates Thingifier docs to point to https://github.com/eviltester/apichallenges
  • keeps the reduced Thingifier reactor green and fixes two small static-analysis issues now exposed by the reduced build

Verification

  • mvn -B clean test
  • mvn -B spotless:check
  • mvn -B -DskipTests verify
  • mvn -B -pl standAloneTodoListManagerRestApiAuto -am test
  • stale-reference search for challengerAuto, ChallengeMain, docker/apichallenges, and uk.co.compendiumdev.challenge

New repo

The extracted API Challenges repo is now at https://github.com/eviltester/apichallenges with Java CI and Docker CI passing on the latest runs.

Copilot AI review requested due to automatic review settings July 16, 2026 22:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@eviltester
eviltester merged commit f4cac7d into master Jul 16, 2026
3 checks passed
@eviltester
eviltester deleted the codex/split-api-challenges-cleanup branch July 16, 2026 22:33
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.

2 participants