Skip to content

IBX-12091: Added frontend-install action for the TypeScript translation extractor - #106

Merged
alongosz merged 1 commit into
mainfrom
IBX-12091-install-typescript-estree
Aug 4, 2026
Merged

alongosz merged 1 commit into
mainfrom
IBX-12091-install-typescript-estree

Conversation

@mikadamczyk

@mikadamczyk mikadamczyk commented Jul 29, 2026

Copy link
Copy Markdown
Contributor
🎫 Issue IBX-12091

Related PRs:

Description:

ibexa/admin-ui extracts translations from TypeScript with a Node script importing @typescript-eslint/typescript-estree, which Composer cannot install. TranslationTest fails in packages shipping .ts/.tsx files (example).
This adds a frontend-install action to be called next to composer-install in those packages. It runs npm ci --ignore-scripts with the lockfile shipped here, installing into vendor/, where Node resolves the package from. composer-install stays untouched and packages that do not call it are unaffected.

For QA:

Documentation:

@mikadamczyk
mikadamczyk marked this pull request as draft July 29, 2026 07:33
@mikadamczyk
mikadamczyk force-pushed the IBX-12091-install-typescript-estree branch 3 times, most recently from 29c6b41 to a3711bd Compare July 29, 2026 09:34
@mikadamczyk mikadamczyk changed the title IBX-12091: Installed typescript-estree for TypeScript translation extraction IBX-12091: Added frontend-install action for the TypeScript translation extractor Jul 29, 2026
@mikadamczyk mikadamczyk self-assigned this Jul 29, 2026
@mikadamczyk
mikadamczyk force-pushed the IBX-12091-install-typescript-estree branch from a3711bd to edb25ba Compare July 29, 2026 09:50
@mikadamczyk
mikadamczyk force-pushed the IBX-12091-install-typescript-estree branch from edb25ba to f11731b Compare July 29, 2026 10:35
@sonarqubecloud

Copy link
Copy Markdown

@mikadamczyk
mikadamczyk requested review from a team July 30, 2026 10:21
@alongosz
alongosz merged commit a162802 into main Aug 4, 2026
2 checks passed
@alongosz
alongosz deleted the IBX-12091-install-typescript-estree branch August 4, 2026 10:53
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.

4 participants