Re: [PR] _frontend/widget: Allow showing the CAS digest of built artifacts [buildstream]

["gtristan (via GitHub)" <gi...@apache.org>]

gtristan commented on code in PR #1994:
URL: https://github.com/apache/buildstream/pull/1994#discussion_r2072581909


##########
man/bst-show.1:
##########
@@ -45,6 +45,7 @@ Show elements in the pipeline
         %{deps}           A list of all dependencies
         %{build-deps}     A list of build dependencies
         %{runtime-deps}   A list of runtime dependencies
+        %{artifact-cas-digest} The CAS digest of the built artifact

Review Comment:
   This change belongs in the docstring of the `show()` command in `_frontend/cli.py`, and the man pages are generated from the frontend docstrings.
   
   Updating the man pages is [documented here](https://docs.buildstream.build/master/hacking/writing_documentation.html#man-pages) (I know, its a bit of an obscure location and the contributing guide is a bit too much and comes from days when we had way too many contributors).
   
   Also the man page generation will not work right now, this is fixed in the `tristan/sboms` branch in 0c2a92d2667f7f88938bea8779d50484ed6b6d94



-- 
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: commits-unsubscribe@buildstream.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org