-
Notifications
You must be signed in to change notification settings - Fork 2
Crowdin POC #153
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: docs/from-gitbook
Are you sure you want to change the base?
Crowdin POC #153
Changes from all commits
e3d7dcc
e690dfb
26e1239
24ab6db
7fa86a7
aaadacb
0b85bb1
6a6f1c1
82337d5
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| name: Setup Node.JS | ||
| description: Setup Node.JS | ||
|
|
||
| inputs: | ||
| package_manager: | ||
| description: 'The package manager to use' | ||
| default: 'npm' | ||
| required: true | ||
|
|
||
| runs: | ||
| using: "composite" | ||
| steps: | ||
| - name: Setup node | ||
| uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5.0.0 | ||
| with: | ||
| node-version-file: '.node-version' | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,56 @@ | ||
| name: Crowdin Action | ||
|
|
||
| on: | ||
| push: | ||
| branches: [ main ] | ||
|
|
||
| jobs: | ||
| synchronize-with-crowdin: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Setup Node.JS | ||
| uses: ./.github/actions/setup-node | ||
|
|
||
| - name: Cache npm dependencies | ||
| id: cache-npm | ||
| uses: actions/cache@v3 | ||
| with: | ||
| path: ~/.npm | ||
| key: ${{ runner.os }}-npm-${{ hashFiles('**/package-lock.json') }} | ||
| restore-keys: | | ||
| ${{ runner.os }}-npm- | ||
|
|
||
| - name: Install dependencies | ||
| run: | | ||
| npm config set cache ~/.npm | ||
| npm ci | ||
|
|
||
| - name: Generate crowdin file | ||
| id: extract_files | ||
| run: npm run generate_file | ||
|
|
||
| - name: crowdin action | ||
| uses: crowdin/github-action@v2 | ||
| with: | ||
| upload_sources: true | ||
| upload_translations: false | ||
| auto_approve_imported: true | ||
| download_translations: false | ||
| create_pull_request: false | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| CROWDIN_PROJECT_ID: ${{ secrets.CROWDIN_PROJECT_ID }} | ||
| CROWDIN_PERSONAL_TOKEN: ${{ secrets.CROWDIN_PERSONAL_TOKEN }} | ||
|
|
||
| - name: Trigger download workflow | ||
| env: | ||
| GH_TOKEN: ${{ secrets.PAT_TOKEN }} | ||
| run: | | ||
|
Comment on lines
+49
to
+52
|
||
| gh workflow run getTranslations.yml \ | ||
| --repo pagopa/devportal-docs-translations \ | ||
| --ref main \ | ||
| -f files='${{ steps.extract_files.outputs.found_files }}' | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| *.idea | ||
| .env | ||
| node_modules |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| 22.18.0 |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,33 @@ | ||||||
| { | ||||||
| "name": "devportal-docs", | ||||||
| "version": "1.0.0", | ||||||
| "description": "This repository contains all some spaces synched from GitBook", | ||||||
|
||||||
| "description": "This repository contains all some spaces synched from GitBook", | |
| "description": "This repository contains some spaces synced from GitBook", |
Copilot
AI
Mar 12, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@types/node is pinned to v25.x while the repo’s runtime Node version is 22.18.0 (see .node-version). Using mismatched Node typings can introduce incorrect type definitions and TS compilation/runtime assumptions. Consider aligning @types/node major to the Node runtime major (22.x) unless you intentionally target Node 25 APIs.
| "@types/node": "^25.2.3", | |
| "@types/node": "^22.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This composite action defines an input
package_manager, but it’s not referenced in any step. Either wire it into the action (e.g., to choose npm/pnpm/yarn setup) or remove the unused input to avoid confusing callers.