Skip to content

SHIP-125: update circleci.bash for GitLab migration#1340

Open
ramandeepromana wants to merge 1 commit into
masterfrom
SHIP-125
Open

SHIP-125: update circleci.bash for GitLab migration#1340
ramandeepromana wants to merge 1 commit into
masterfrom
SHIP-125

Conversation

@ramandeepromana

Copy link
Copy Markdown
Contributor

Summary

  • Update CI clone URL from GitHub to GitLab for the GitHub-to-GitLab VCS migration
  • Replace GitHub API tag lookup with local git tag -l (simpler, no auth, no rate limits)
  • Phase 1 of migration: CI-critical changes only, merged before mirror break

Changes

scripts/circleci.bash

  • Clone URL (line 5): github.com/2600hz/monster-ui -> gitlab.com/oomaforbin/oomacorp/2600hz/monster-ui -- points CI at GitLab as the source of truth
  • Tag lookup (lines 13-17): Replace curl https://api.github.com/repos/2600hz/monster-ui/git/refs/tags | jq with git tag -l | sort --version-sort | grep | tail -1 -- uses locally fetched tags instead of GitHub API, eliminating auth requirements and rate limit concerns

Test Plan

  • Verify git tag -l | sort --version-sort | grep "^5.4" | grep -E '^[0-9]+\.[0-9]+\.[0-9]+' | tail -1 returns the correct latest tag for a release branch
  • CI build on this branch should clone from GitLab and resolve the correct core version tag

Ticket

SHIP-125: Update the GitHub repos to GitLab in all monster-ui scripts on monster-ui repo
https://oomacorp.atlassian.net/browse/SHIP-125

- Clone URL: github.com/2600hz -> gitlab.com/oomaforbin/oomacorp/2600hz
- Tag lookup: replace GitHub API call with local git tags (simpler,
  no auth needed, no rate limits)
@ramandeepromana ramandeepromana self-assigned this Feb 24, 2026

@guillegr123 guillegr123 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

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