Skip to content

fix: translation bundler logic#22459

Merged
hbjORbj merged 3 commits intomainfrom
fix/translation-bundler-logic
Jul 14, 2025
Merged

fix: translation bundler logic#22459
hbjORbj merged 3 commits intomainfrom
fix/translation-bundler-logic

Conversation

@hbjORbj
Copy link
Contributor

@hbjORbj hbjORbj commented Jul 13, 2025

What does this PR do?

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Please use the latest Vercel preview and test please 🙏.

@graphite-app graphite-app bot requested a review from a team July 13, 2025 22:35
@keithwillcode keithwillcode added core area: core, team members only foundation labels Jul 13, 2025
@dosubot dosubot bot added the 🐛 bug Something isn't working label Jul 13, 2025
cubic-dev-ai[bot]
cubic-dev-ai bot previously approved these changes Jul 13, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic reviewed 1 file and found no issues. Review PR in cubic.dev.

@vercel
Copy link

vercel bot commented Jul 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Jul 13, 2025 10:38pm
cal-eu ⬜️ Ignored (Inspect) Visit Preview Jul 13, 2025 10:38pm

@delve-auditor
Copy link

delve-auditor bot commented Jul 13, 2025

No security or compliance issues detected. Reviewed everything up to 9f08530.

Security Overview
  • 🔎 Scanned files: 3 changed file(s)
Detected Code Changes
Change Type Relevant files
Refactor ► packages/lib/server/translationBundler.ts
    Update monorepo root directory detection logic
Other ► package.json
    Remove ts-node dependency
► yarn.lock
    Update dependency lock file

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

@hbjORbj hbjORbj enabled auto-merge (squash) July 13, 2025 22:37
@hbjORbj hbjORbj requested review from keithwillcode and removed request for a team July 13, 2025 22:37
@socket-security
Copy link

socket-security bot commented Jul 13, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedmemory-cache@​0.2.010010010076100
Addedbcryptjs@​2.4.31001001008080

View full report

@graphite-app
Copy link

graphite-app bot commented Jul 13, 2025

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (07/13/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add ready-for-e2e label" took an action on this PR • (07/13/25)

1 label was added to this PR based on Keith Williams's automation.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 13, 2025

E2E results are ready!

@hbjORbj hbjORbj merged commit 66397a6 into main Jul 14, 2025
109 of 116 checks passed
@hbjORbj hbjORbj deleted the fix/translation-bundler-logic branch July 14, 2025 02:09
alishaz-polymath pushed a commit that referenced this pull request Jul 16, 2025
* fix logic in translationBundler

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

Labels

🐛 bug Something isn't working core area: core, team members only foundation ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants