Skip to content

Conversation

@eepifanova
Copy link
Contributor

The reusable workflow to build nginx.org
Should be run on nginx/nginx.org

@eepifanova eepifanova requested a review from a team as a code owner March 3, 2025 18:39
@oxpa
Copy link

oxpa commented Mar 3, 2025

LGTM

oxpa
oxpa previously approved these changes Mar 3, 2025
The reusable workflow to build nginx.org
Should be run on nginx/nginx.org
@eepifanova
Copy link
Contributor Author

@thresheek plz review
I've added she for actions/checkout and moved the target to the repo var

@eepifanova
Copy link
Contributor Author

@thresheek
Checked on the following workflow - completed successfully
https://docs.github.com/en/actions/sharing-automations/reusing-workflows#limitations

on:
  push:
    branches:
      - main
    paths:
      - 'binary/**'
      - 'css/**'
      - 'dtd/**'
      - 'img/**'
      - 'sources/**'
      - 'text/**'
      - 'tools/**'
      - 'xml/**'
      - 'xsls/**'
      - 'xslt/**'
      - 'yaml/**'
  
permissions:
  contents: read
  id-token: write

jobs:
  build:
    name: build site
    uses: nginx/ci-self-hosted/.github/workflows/nginx.org-build.yml@nginx.org        `

@thresheek thresheek merged commit 666487f into main Mar 24, 2025
1 check passed
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