Skip to content

add translated deno.mdx in it/guides/deploy#14183

Open
uniboxx wants to merge 10 commits into
withastro:mainfrom
uniboxx:i18n-it-deno-mdx
Open

add translated deno.mdx in it/guides/deploy#14183
uniboxx wants to merge 10 commits into
withastro:mainfrom
uniboxx:i18n-it-deno-mdx

Conversation

@uniboxx

@uniboxx uniboxx commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Description (required)

Translation of deno.mdx in it/guides/deploy

My discord account is uniboxx_68360

@astrobot-houston

astrobot-houston commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
en/guides/deploy/deno.mdx Source changed, localizations will be marked as outdated.
it/guides/deploy/deno.mdx Localization added, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@github-actions github-actions Bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Jun 30, 2026
@github-actions

github-actions Bot commented Jun 30, 2026

Copy link
Copy Markdown

Preview deployment

✅ Deployment complete!

@yanthomasdev yanthomasdev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @uniboxx, I made two comments for you to consider 🙌

Comment on lines +198 to +217
permissions:
id-token: write # Needed for auth with Deno Deploy
contents: read # Needed to clone the repository

steps:
- name: Clone repository
uses: actions/checkout@v6

# Not using npm? Change `npm ci` to `yarn install` or `pnpm i`
- name: Install dependencies
run: npm ci

# Not using npm? Change `npm run build` to `yarn build` or `pnpm run build`
- name: Build Astro
run: npm run build

- name: Upload to Deno Deploy
uses: denoland/deployctl@v1
with:
project: my-deno-project # TODO: replace with Deno Deploy project name

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think you missed translating these comments @uniboxx

Comment on lines +236 to +255
id-token: write # Needed for auth with Deno Deploy
contents: read # Needed to clone the repository

steps:
- name: Clone repository
uses: actions/checkout@v6

# Not using npm? Change `npm ci` to `yarn install` or `pnpm i`
- name: Install dependencies
run: npm ci

# Not using npm? Change `npm run build` to `yarn build` or `pnpm run build`
- name: Build Astro
run: npm run build

- name: Upload to Deno Deploy
uses: denoland/deployctl@v1
with:
project: my-deno-project # TODO: replace with Deno Deploy project name
entrypoint: dist/server/entry.mjs

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Here as well

@yanthomasdev

Copy link
Copy Markdown
Member

@uniboxx you updated it in the English file instead of the Italian one 😆, it happens! Could you fix it when you have the time? Thanks.

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

Labels

i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants