gtristan commented on code in PR #2035:
URL: https://github.com/apache/buildstream/pull/2035#discussion_r2199932939
##########
src/buildstream/_frontend/app.py:
##########
@@ -302,6 +304,10 @@ def initialized(self, *, session_name=None):
#
self.stream.set_project(self.project)
+ # Initialize the inspector
+ if self._session_name == "Inspect":
Review Comment:
I don't think `bst inspect` should have a session name.
The session name is a UI detail used to determine whether the print an
elaborate heading about buildstream version and project details, and whether to
time the overall session encapsulated in a START / SUCCESS / FAILURE block,
these are more appropriate for work related commands like `bst build`, `bst
source fetch`, `bst artifact push`, `bst source track` etc (i.e. *sessions*).
--
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]
