This PR will consolidate the Restoring R Environment and Restoring Python Environment phases for the Publishing log into one single stage.
It also:
updates the status of the stages so that when they are active, the title is active vs. a passive statement.
updates the label of a skipped stage of a successful deployment, to show the step as being skipped and with a checkmark. (See captured image below)
Resolves #586
Type of Change
[ ] Bug Fix
[x] New Feature
[ ] Breaking Change
[ ] Documentation
[ ] Refactor
[ ] Tooling
Approach
The two sets of messages for Python and R environment restoration now will also produce a new generic environment restoration set. This is done as the Python and R messages are received. To the rest of the UX code, these new events might as well have been sent from the backend. This allows for registration as expected as well as valid typing (where we are using it).
Additional strings were added to display in the tree as labels while the stage was considered active vs. inactive. This provides for the correct tense.
Showing skipped step (this is for Quarto Content as an example)
Automated Tests
Directions for Reviewers
Deploy R, Python and Quarto content and confirm that the publisher log shows realtime output as expected as well as the static view when deployment has finished (with either success or failure).
Checklist
[ ] I have updated CHANGELOG.md to cover notable changes.
Intent
This PR will consolidate the Restoring R Environment and Restoring Python Environment phases for the Publishing log into one single stage.
It also:
Resolves #586
Type of Change
Approach
The two sets of messages for Python and R environment restoration now will also produce a new generic environment restoration set. This is done as the Python and R messages are received. To the rest of the UX code, these new events might as well have been sent from the backend. This allows for registration as expected as well as valid typing (where we are using it).
Additional strings were added to display in the tree as labels while the stage was considered active vs. inactive. This provides for the correct tense.
This results in the following interfaces:
For R Content https://github.com/posit-dev/publisher/assets/17675905/f2f7edcf-2967-40d2-aa78-d578ad1c5733
For Python Content https://github.com/posit-dev/publisher/assets/17675905/1efe3b8a-7185-4731-a6a8-5d8fa5521341
Showing skipped step (this is for Quarto Content as an example)![2024-05-31 at 11 57 AM](https://github.com/posit-dev/publisher/assets/17675905/cb05a9bc-ff2f-4d57-a496-70d69783d8d8)
Automated Tests
Directions for Reviewers
Deploy R, Python and Quarto content and confirm that the publisher log shows realtime output as expected as well as the static view when deployment has finished (with either success or failure).
Checklist