Skip to content
This repository was archived by the owner on Apr 4, 2023. It is now read-only.

Conversation

@azatsarynnyy
Copy link
Member

@azatsarynnyy azatsarynnyy commented Jun 15, 2021

…s when no code is changed

Signed-off-by: Artem Zatsarynnyi <azatsary@redhat.com>
…s when no code is changed

Signed-off-by: Artem Zatsarynnyi <azatsary@redhat.com>
@codecov
Copy link

codecov bot commented Jun 15, 2021

Codecov Report

Merging #1141 (5487113) into master (02d0cb0) will decrease coverage by 23.88%.
The diff coverage is 57.13%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master    #1141       +/-   ##
===========================================
- Coverage   56.46%   32.58%   -23.89%     
===========================================
  Files          47      286      +239     
  Lines        2088     9714     +7626     
  Branches      346     1472     +1126     
===========================================
+ Hits         1179     3165     +1986     
- Misses        898     6451     +5553     
- Partials       11       98       +87     
Impacted Files Coverage Δ
...theia-about/src/browser/about-che-theia-dialog.tsx 0.00% <ø> (ø)
...e-sync-tracker/src/browser/sync-process-tracker.ts 0.00% <0.00%> (ø)
...ovisioner/src/node/git-configuration-controller.ts 0.00% <0.00%> (ø)
...rowser/src/browser/che-mini-browser-environment.ts 0.00% <0.00%> (ø)
...mini-browser/src/node/che-mini-browser-endpoint.ts 0.00% <0.00%> (ø)
...se-che-theia-plugin-ext/src/common/che-protocol.ts 100.00% <ø> (ø)
...e-che-theia-plugin-ext/src/plugin/che-workspace.ts 56.66% <0.00%> (ø)
...lugin-remote/src/browser/che-plugin-file-system.ts 0.00% <0.00%> (ø)
...emote/src/browser/plugin-remote-frontend-module.ts 0.00% <0.00%> (ø)
...e-theia-plugin-remote/src/common/che-plugin-uri.ts 0.00% <0.00%> (ø)
... and 351 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 33170cf...5487113. Read the comment docs.

@azatsarynnyy azatsarynnyy marked this pull request as ready for review June 15, 2021 14:39
@azatsarynnyy azatsarynnyy requested a review from benoitf as a code owner June 15, 2021 14:39
@che-bot
Copy link
Contributor

che-bot commented Jun 15, 2021

✅ E2E Happy path tests succeed 🎉

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia quay.io/crw_pr/che-theia:1141
che-theia-endpoint-runtime-binary quay.io/crw_pr/che-theia-endpoint-runtime-binary:1141

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@che-bot
Copy link
Contributor

che-bot commented Jun 15, 2021

✅ E2E Happy path tests succeed 🎉

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia quay.io/crw_pr/che-theia:1141
che-theia-endpoint-runtime-binary quay.io/crw_pr/che-theia-endpoint-runtime-binary:1141

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@azatsarynnyy azatsarynnyy merged commit 340363a into master Jun 15, 2021
@azatsarynnyy azatsarynnyy deleted the pr-chk-impv branch June 15, 2021 18:04
@che-bot che-bot added this to the 7.32 milestone Jun 15, 2021
dmytro-ndp pushed a commit that referenced this pull request Jul 21, 2021
… changed (#1141)

* build(pr-check): Split the GH Workflow to avoid running the build jobs when no code is changed

Signed-off-by: Artem Zatsarynnyi <azatsary@redhat.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants