Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: use description field in emui logs #2199

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

Dartoxian
Copy link
Contributor

Description:

This PR uses the new description field in the enclave logs viewer. By default this field is now used to render starlark instructions, rather than the raw field (from #2147 ).

Demo

description-toggle.mp4

Is this change user facing?

Yes

References (if applicable):

  • briefed on slack.

@Dartoxian Dartoxian requested a review from adschwartz February 21, 2024 09:35
@Dartoxian Dartoxian added this pull request to the merge queue Feb 21, 2024
Merged via the queue into main with commit ad00ee1 Feb 21, 2024
43 checks passed
@Dartoxian Dartoxian deleted the bgazzard/enclave-logs-description branch February 21, 2024 15:07
github-merge-queue bot pushed a commit that referenced this pull request Feb 21, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.87.1](0.87.0...0.87.1)
(2024-02-21)


### Features

* use description field in emui logs
([#2199](#2199))
([ad00ee1](ad00ee1))


### Bug Fixes

* admonition in github guide
([#2195](#2195))
([c2d4d42](c2d4d42))
* Fix the connect to enclave CLI commands in the EM UI
([#2203](#2203))
([2218f4b](2218f4b))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: kurtosisbot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants