Adding render option to the actions "render" and "publish" and cleaning code#108
Adding render option to the actions "render" and "publish" and cleaning code#108m-fehr wants to merge 1 commit intoquarto-dev:mainfrom
Conversation
|
Thanks. Do you find an explicit profile option to the action easier than using - name: Deploy gh-pages Preview
uses: quarto-dev/quarto-actions/publish@v2
with:
target: gh-pages
env:
QUARTO_PROFILE: previewAsking because adding |
|
I believe it is more intuitive considering that "--to" is also an explicit profile option. When I reviewed the README for the render action, I naturally assumed there would be a corresponding render option. However, you could leave it as it is now, but it should be added to the README file. |
|
I have added mention of @cscheid are you in favor of adding a new with:
profile: previewoption in the action instead of just setting env:
QUARTO_PROFILE: previewJust to get another view on this. Thanks ! |
|
I don't have a strong opinion either way. |
This pull request addresses the feature in #102
What I did:
Introduced an optional profile input argument for specifying the profile to give to the render command.
To simplify the conditional logic the command strings are now dynamically build.
Moved the Git configuration to a separate step for clarity.