Skip to content

CORE-2517 Post-migration updates for tradingtechnologies org - #2824

Open
anikettt2499 wants to merge 1 commit into
mainfrom
topic/post-migration
Open

anikettt2499 wants to merge 1 commit into
mainfrom
topic/post-migration

Conversation

@anikettt2499

Copy link
Copy Markdown

Summary

  • Step 1 (CircleCI): No change needed — opengamma-orb is not referenced in .circleci/config.yml.
  • Step 2 (GitHub workflows): Created all 5 standard workflows (none existed before): pr-fix.yml, pr-review.yml, pr-review-stats.yml, tt-margin-pr-automation.yml, tt-margin-pr-automation-target.yml.
  • Step 3 (Dependabot): No change needed — no Artifactory registry entry in .github/dependabot.yml.
  • Step 4 (CODEOWNERS): No change needed — only entry is @jodastephen (a username, not an OpenGamma team slug).
  • Step 5 (Mergify): Deleted .github/mergify.yml.

Repo admin steps (in order)

  1. Remove Mergify Merge Protections from the required status checks on main.
  2. Merge this PR.
  3. Add jira / Ensure JIRA reference as a required status check on main.

Automation workflow requirements

The two tt-margin-pr-automation workflows require:

  • The tt-margin-bot GitHub App is installed on the repository (Contents and Pull requests write permissions).
  • tt-margin-bot is added to "Restrict who can push" on main if that restriction is enabled.
  • Allow auto-merge is turned on in repository settings.

What Mergify did that is not replaced

  • Merge queue ordering and urgent/dependency priorities.
  • Converting WIP titles to draft (a non-draft WIP PR with the auto-merge label will now auto-merge once approved).
  • Stale PR comments.
  • Size labels.

Jira: CORE-2517

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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