Skip to content

Upgrade traceviewer components, tsp-typescript-client and timeline-chart#330

Merged
bhufmann merged 1 commit into
eclipse-cdt-cloud:masterfrom
bhufmann:upgade-components
Jun 26, 2025
Merged

Upgrade traceviewer components, tsp-typescript-client and timeline-chart#330
bhufmann merged 1 commit into
eclipse-cdt-cloud:masterfrom
bhufmann:upgade-components

Conversation

@bhufmann
Copy link
Copy Markdown
Contributor

What it does

Upgrade traceviewer components, tsp-typescript-client and timeline-chart

How to test

Successful CI. Open some traces and views for sanity checks.

Follow-ups

N/A

Review checklist

  • As an author, I have thoroughly tested my changes and carefully followed the instructions in this template

- traceviewer-base and traceviewer-react-components: v0.9.0
eclipse-cdt-cloud/theia-trace-extension#1201

- tsp-typescript-client: v0.7.0
eclipse-cdt-cloud/tsp-typescript-client#131

- timeline-chart: v0.4.2
eclipse-cdt-cloud/timeline-chart#320

Signed-off-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
@marcdumais-work
Copy link
Copy Markdown
Contributor

There's currently an issue, running dash-licenses, the tool behind the license check workflow. I think it's low-risk to proceed anyway - we've successfully run it against the new trace components that are updated in this PR here, in their respective repos.

Copy link
Copy Markdown
Contributor

@marcdumais-work marcdumais-work left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks for this contribution!

@marcdumais-work
Copy link
Copy Markdown
Contributor

OOPS - file RELEASE needs to be updated with the new version

Copy link
Copy Markdown
Contributor

@marcdumais-work marcdumais-work left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We forgot to update file RELEASE

Copy link
Copy Markdown
Contributor

@marcdumais-work marcdumais-work left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad - this is the component update - a release PR should follow

@bhufmann bhufmann merged commit cc47ed4 into eclipse-cdt-cloud:master Jun 26, 2025
7 of 9 checks passed
@bhufmann bhufmann deleted the upgade-components branch June 26, 2025 19:21
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