Skip to content

Conversation

@srl295
Copy link
Member

@srl295 srl295 commented May 22, 2025

  • also, update about page to link to CLDR-JSON

CLDR-18512

  • This PR completes the ticket.

ALLOW_MANY_COMMITS=true

- also, update about page to link to CLDR-JSON
@srl295 srl295 self-assigned this May 22, 2025
@srl295 srl295 requested review from AEApple, btangmu and macchiati May 22, 2025 18:56
@github-actions
Copy link

✨ This commit deployed to https://68e96385.cldr.pages.dev
✨ This PR deployed to https://pr4734.cldr.pages.dev

Changed Pages in the last commit:

@github-actions github-actions bot added the Site Update Updates docs/site label May 22, 2025
@srl295
Copy link
Member Author

srl295 commented May 22, 2025

OK, the PR side works… think its likely the production side will also

accountId: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
command: pages deploy ./_site --project-name=cldr --branch pr${{ github.event.pull_request.number }}
# use '--branch pr####' (pr###.cldr.pages.dev) for PRs, otherwise goes to cldr.unicode.org (production)
command: pages deploy ./_site --project-name=cldr ${{ github.event_name == 'push' && '' || format('--branch pr{0}', github.event.pull_request.number) }}
Copy link
Member Author

Choose a reason for hiding this comment

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

for push (main) use no options ''

for a PR use --branch pr####

@srl295 srl295 merged commit fe0d836 into main May 22, 2025
19 checks passed
@srl295 srl295 deleted the srl295/cldr-18512/fix-site-deploy branch May 22, 2025 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Site Update Updates docs/site

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants