eclipse-che / che-theia

Eclipse Public License 2.0
124 stars 110 forks source link

Align Theia version in the dependencies #1276

Closed azatsarynnyy closed 2 years ago

azatsarynnyy commented 2 years ago

Signed-off-by: Artem Zatsarynnyi azatsary@redhat.com

What does this PR do?

Set the Theia dependencies versions for 7.40 the same as it's for 7.38. As both versions depend on the same upstream commit hash, so no reason tp depend on another version of the npm packages.

Screenshot/screencast of this PR

What issues does this PR fix or reference?

An attempt to fix https://issues.redhat.com/browse/CRW-2566

How to test this PR?

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Happy Path Channel

HAPPY_PATH_CHANNEL=stable

azatsarynnyy commented 2 years ago

[crw-ci-test --rebuild]

Thanks @dmytro-ndp for looking at it, but AFAIK, it works for main branch only. Correct?

upd: not relevant, as Dmytro has removed his comment :man_shrugging:

dmytro-ndp commented 2 years ago

@azatsarynnyy : you are right. I should had looked at branch to merge before triggering PR check. It's now working for main branch only.