slachiewicz commented on PR #71:
URL: https://github.com/apache/maven-doxia-site/pull/71#issuecomment-5230880946

   Correction to my previous comment: I was wrong that the site step cannot be 
suppressed. It is gated, and there is an input for it:
   
   ```yaml
         - name: Build Maven Site
           run: ./mvnw ... ${{ inputs.ff-site-goal }}
           if: inputs.ff-site-run
   ```
   
   `ff-site-run` defaults to `true`, which is why this PR switched the step on, 
but `ff-site-run: false` would have kept it off. So a caller-side configuration 
does exist here:
   
   ```yaml
       with:
         matrix-enabled: false
         ff-goal: 'clean site'
         ff-site-run: false
   ```
   
   I am still leaving this closed, because apache/maven-gh-actions-shared#300 
fixes the gating centrally and leaves this repository's configuration 
untouched, which is the smaller change here. Happy to reopen with the 
three-input version if the caller-side route is preferred.
   
   The `site:stage` failure this run surfaced is unrelated to either fix and 
still worth a look on its own:
   
   ```
   Error copying directory structure: Unknown file type: 
.../target/site/doxia-tools-archives
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to