Skip to content
This repository was archived by the owner on Jun 23, 2025. It is now read-only.

Adopted fix for angular universal support #1747

Merged
8 commits merged into from
Nov 12, 2019
Merged

Adopted fix for angular universal support #1747

8 commits merged into from
Nov 12, 2019

Conversation

pertsenga
Copy link
Contributor

@pertsenga pertsenga commented Oct 14, 2019

Adopted #1554 for #1052 fix

@codecov
Copy link

codecov bot commented Oct 14, 2019

Codecov Report

Merging #1747 into master will increase coverage by 0.13%.
The diff coverage is 88.88%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1747      +/-   ##
==========================================
+ Coverage   47.84%   47.97%   +0.13%     
==========================================
  Files          47       47              
  Lines        1992     1999       +7     
  Branches      176      177       +1     
==========================================
+ Hits          953      959       +6     
- Misses       1035     1036       +1     
  Partials        4        4
Impacted Files Coverage Δ
packages/core/directives/map.ts 72.19% <88.88%> (+0.52%) ⬆️

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 a1a17a6...0d8355f. Read the comment docs.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

we left rollup for ng-pacakgr

@mehrad-rafigh
Copy link

@doom777 Any chance we can get this merged?

@ghost
Copy link

ghost commented Oct 28, 2019

@mehrad-rafigh yes, but there are many things that need to be changed

@mehrad-rafigh
Copy link

@doom777 Thank you for your answer. Would you please mind pointing out, what still would need to be changed?

@pertsenga pertsenga marked this pull request as ready for review October 29, 2019 04:45
@pertsenga pertsenga changed the title Laurent fix universal pr adopted Adopted fix for angular universal support Oct 29, 2019
@ghost
Copy link

ghost commented Nov 5, 2019

Ok, so remove tsconfig changes, and merge, and I'll do a final check

@ghost ghost merged commit 7e408e9 into sebholstein:master Nov 12, 2019
@mehrad-rafigh
Copy link

Is this going to be released soon? Eagerly waiting for it :)

@ghost
Copy link

ghost commented Nov 16, 2019

Whenever @SebastianM is back in contact

This pull request was closed.
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