Skip to content
This repository has been archived by the owner on Mar 8, 2023. It is now read-only.

MAPSJS-3047, Split theme to stable and unstable parts #2331

Merged
merged 1 commit into from
Nov 16, 2021
Merged

Conversation

AndriiHeonia
Copy link
Contributor

Signed-off-by: Andrii Heonia andrii.heonia@here.com

@AndriiHeonia AndriiHeonia force-pushed the MAPSJS-3047 branch 3 times, most recently from bd26de8 to 0b1114f Compare October 29, 2021 13:59
@codecov
Copy link

codecov bot commented Oct 29, 2021

Codecov Report

Merging #2331 (c3ee308) into master (eb777fe) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2331   +/-   ##
=======================================
  Coverage   68.30%   68.30%           
=======================================
  Files         319      319           
  Lines       27815    27815           
  Branches     6237     6237           
=======================================
+ Hits        18999    19000    +1     
+ Misses       8816     8815    -1     
Impacted Files Coverage Δ
@here/harp-mapview/lib/MapMaterialAdapter.ts 92.85% <0.00%> (+0.71%) ⬆️

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 eb777fe...c3ee308. Read the comment docs.

@ThFischer ThFischer marked this pull request as ready for review November 2, 2021 11:17
ThFischer
ThFischer previously approved these changes Nov 2, 2021
@gairik
Copy link
Contributor

gairik commented Nov 11, 2021

@AndriiHeonia I just updated the CI to use node 14 from node 12. So you see those new tests. You can rebase your changes to add the new commits and push once more.
The tests on node 12 is no longer compulsory but the tests on v14 are.

Signed-off-by: Andrii Heonia <andrii.heonia@here.com>
@dbacarel dbacarel merged commit 9e9d58e into master Nov 16, 2021
@dbacarel dbacarel deleted the MAPSJS-3047 branch November 16, 2021 07:52
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.

None yet

4 participants