Skip to content

chore(ci): fix e2e workflow auth for private git dependency#1866

Merged
ekremney merged 1 commit intomainfrom
chore/fix-e2e-ci-auth
Feb 25, 2026
Merged

chore(ci): fix e2e workflow auth for private git dependency#1866
ekremney merged 1 commit intomainfrom
chore/fix-e2e-ci-auth

Conversation

@ekremney
Copy link
Member

Summary

  • Add MYSTICAT_DATA_SERVICE_REPO_READ_TOKEN env var to the setup-node-npm step in the e2e-tests workflow
  • Add persist-credentials: 'false' to the checkout step to prevent the GITHUB_TOKEN credential helper from overriding the git URL rewrite

The e2e workflow was failing on npm ci because it couldn't resolve the private mysticat-data-service git dependency. The CI workflow (ci.yaml) already has both of these configured correctly — this brings the e2e workflow in line.

Test plan

  • E2e workflow runs successfully (next scheduled run or manual trigger)

🤖 Generated with Claude Code

The e2e-tests workflow was missing the MYSTICAT_DATA_SERVICE_REPO_READ_TOKEN
env var on the setup-node-npm step, causing npm ci to fail when resolving
the private mysticat-data-service git dependency. Also added
persist-credentials: false to prevent the GITHUB_TOKEN credential helper
from overriding the git URL rewrite.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ekremney ekremney requested a review from blefebvre February 25, 2026 13:33
@ekremney ekremney self-assigned this Feb 25, 2026
@ekremney ekremney merged commit 09104ac into main Feb 25, 2026
18 checks passed
@ekremney ekremney deleted the chore/fix-e2e-ci-auth branch February 25, 2026 15:35
@codecov
Copy link

codecov bot commented Feb 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@solaris007
Copy link
Member

🎉 This PR is included in version 1.324.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants