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

Adopt tsp-typescript-client version 0.5.1 and pin inversify to 6.0.3 #1141

Merged
merged 2 commits into from
Nov 11, 2024

Conversation

bhufmann
Copy link
Collaborator

@bhufmann bhufmann commented Nov 11, 2024

  • Use resolution to pin inversify to v6.0.3 to avoid docker build errors
    The docker example doesn't have yarn.lock file and hence the builds starts fresh. The new inversify version v6.1.x cause a build error.
  • Adopt tsp-typescript-client version 0.5.1

Signed-off-by: Bernd Hufmann [email protected]

@bhufmann
Copy link
Collaborator Author

Licences check fails. Ticket has been submitted for the new version: https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/17159

@marcdumais-work
Copy link
Contributor

I think the Docker CI failure is most likely unrelated to this PR and probably caused by pulling newer versions of some dependencies. This happens because that example has no lockfile.

PatrickTasse
PatrickTasse previously approved these changes Nov 11, 2024
@bhufmann bhufmann changed the title Adopt tsp-typescript-client version 0.5.1 Adopt tsp-typescript-client version 0.5.1 and pin inversify to 6.0.3 Nov 11, 2024
@bhufmann
Copy link
Collaborator Author

bhufmann commented Nov 11, 2024

I think the Docker CI failure is most likely unrelated to this PR and probably caused by pulling newer versions of some dependencies. This happens because that example has no lockfile.

I added a commit to pin the inversify version to 6.0.3 to avoid the error. (see recommendation here eclipse-theia/theia#14431)

The docker example doesn't have yarn.lock file and hence the builds
starts fresh. The new inversify version v6.1.x cause a build error.

Signed-off-by: Bernd Hufmann <[email protected]>
@@ -14,6 +14,7 @@
}
}
},

Copy link
Contributor

Choose a reason for hiding this comment

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

added newline on purpose?

Copy link
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 - one minor comment that you may address if you want

@bhufmann bhufmann merged commit bfb8c63 into eclipse-cdt-cloud:master Nov 11, 2024
6 of 7 checks passed
@bhufmann bhufmann deleted the upgrade branch November 11, 2024 22:04
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.

3 participants