Skip to content

chore: update code-connect yaml#837

Merged
rohanchkrabrty merged 1 commit into
mainfrom
worktree-bug-cc-action
Jun 10, 2026
Merged

chore: update code-connect yaml#837
rohanchkrabrty merged 1 commit into
mainfrom
worktree-bug-cc-action

Conversation

@rohanchkrabrty

Copy link
Copy Markdown
Contributor

Summary

  • Update code-connect yaml to fix env lookup

@vercel

vercel Bot commented Jun 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
apsara Ready Ready Preview, Comment Jun 10, 2026 6:54am

@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 47417e1d-1368-449e-9bd9-9923603f9528

📥 Commits

Reviewing files that changed from the base of the PR and between 60979a8 and 582ce5d.

📒 Files selected for processing (1)
  • .github/workflows/code-connect.yaml

📝 Walkthrough

Walkthrough

This pull request updates the code-connect GitHub Actions workflow to source Figma configuration URLs from secrets instead of public variables. Across three workflow steps—"Mask Figma file keys", "Generate icon mappings", and "Update Figma config"—the FIGMA_FILE_URL and FIGMA_ICONS_FILE_URL environment variables are switched from ${{ vars.* }} to ${{ secrets.* }}. This change ensures these values are properly masked in workflow logs and treated as sensitive data.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'chore: update code-connect yaml' is vague and does not clearly describe the main change—switching Figma URL secrets from vars to secrets for masking and publishing. Revise the title to be more specific, such as 'chore: move Figma URLs from vars to secrets in code-connect workflow' to clarify the purpose of the changes.
✅ Passed checks (4 passed)
Check name Status Explanation
Description check ✅ Passed The description 'Update code-connect yaml to fix env lookup' is vague but is related to the changeset, which involves fixing environment variable lookups by switching from vars to secrets.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@rohanchkrabrty rohanchkrabrty self-assigned this Jun 10, 2026
@rohanchkrabrty rohanchkrabrty merged commit 15e03b2 into main Jun 10, 2026
5 checks passed
@rohanchkrabrty rohanchkrabrty deleted the worktree-bug-cc-action branch June 10, 2026 06:54
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