Skip to content

move plugin-seeds repo to dotcms-community/plugin-examples#35171

Merged
wezell merged 1 commit intomainfrom
issue-35170-move-plugin-seeds
Apr 1, 2026
Merged

move plugin-seeds repo to dotcms-community/plugin-examples#35171
wezell merged 1 commit intomainfrom
issue-35170-move-plugin-seeds

Conversation

@wezell
Copy link
Copy Markdown
Member

@wezell wezell commented Apr 1, 2026

ref: #35170

Moves repo references

This PR fixes: #35170

@wezell wezell requested a review from a team as a code owner April 1, 2026 13:37
@github-actions github-actions Bot added Area : CI/CD PR changes GitHub Actions/workflows Area : Documentation PR changes documentation files labels Apr 1, 2026
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Apr 1, 2026

Claude finished @wezell's task in 49s —— View job


I'll analyze this and get back to you.

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Apr 1, 2026

Claude finished @wezell's task in 1m 21s —— View job


PR Review

  • Gather context and analyze diff

Two issues worth flagging:

1. Cross-org token permissions — verify CI_MACHINE_TOKEN has access

In .github/workflows/cicd_comp_release-phase.yml (line ~222) and legacy-release_maven-release-process.yml (line ~360), the dispatch target changes from dotCMS/plugin-seeds to dotcms-community/plugin-examples — a different GitHub org. The CI_MACHINE_TOKEN secret must have repo (or actions:write) permissions on dotcms-community/plugin-examples for the dispatch to succeed. If the token is scoped to the dotCMS org only, the release workflow will silently fail at the plugin-update step (the curl returns a non-200 status that is logged but the step may not fail the job depending on error handling). Confirm the token has cross-org access before merging.

2. Unrelated URL change in docs/README.txt

The diff also changes http://dotcms.com/http://dev.dotcms.com/. This is unrelated to the plugin-seeds migration stated in the PR description. Pointing public docs at a dev subdomain looks wrong — verify this is intentional.

Everything else (doc link updates, trailing newline fixes) is clean.

@wezell wezell added this pull request to the merge queue Apr 1, 2026
Merged via the queue into main with commit e746462 Apr 1, 2026
50 checks passed
@wezell wezell deleted the issue-35170-move-plugin-seeds branch April 1, 2026 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI: Safe To Rollback Area : CI/CD PR changes GitHub Actions/workflows Area : Documentation PR changes documentation files

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Move plugin-seeds to plugin-examples

2 participants