Merge pull request #303 from rstudio/bcwu-update_write-manifest

fix write-manifest-notebook not using hide input

Author: bcwu

CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.12.1] - 2022-11-07
+
+### Changed
+- Updated actions.py to reuse code in main, minus the CLI parts. As a result deploy_jupyter_notebook and deploy_by_manifest had their return signatures changed. They now return None.
+
 ## [1.12.0] - 2022-10-26
 
 ### Added

rsconnect/bundle.py

@@ -1255,6 +1255,14 @@ def write_notebook_manifest_json(
             raise RSConnectException('Could not determine the app mode from "%s"; please specify one.' % extension)
 
     manifest_data = make_source_manifest(app_mode, environment, file_name, None, image)
+    if hide_all_input or hide_tagged_input:
+        if "jupyter" not in manifest_data:
+            manifest_data["jupyter"] = dict()
+        if hide_all_input:
+            manifest_data["jupyter"]["hide_all_input"] = True
+        if hide_tagged_input:
+            manifest_data["jupyter"]["hide_tagged_input"] = True
+
     manifest_add_file(manifest_data, file_name, directory)
     manifest_add_buffer(manifest_data, environment.filename, environment.contents)
 

rsconnect/main.py

@@ -1283,7 +1283,7 @@ def write_manifest():
     is_flag=True,
     help='Force generating "requirements.txt", even if it already exists.',
 )
-@click.option("--hide-all-input", help="Hide all input cells when rendering output")
+@click.option("--hide-all-input", is_flag=True, default=None, help="Hide all input cells when rendering output")
 @click.option("--hide-tagged-input", is_flag=True, default=None, help="Hide input code cells with the 'hide_input' tag")
 @click.option("--verbose", "-v", "verbose", is_flag=True, help="Print detailed messages")
 @click.option(