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

Adopted fix for angular universal support #1747

Merged
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 force-pushed the laurent-fix-universal-pr-adopted branch from 6daff6c to 7e8f0ca Compare October 29, 2019 04:00
@pertsenga pertsenga force-pushed the laurent-fix-universal-pr-adopted branch from 7e8f0ca to 1cde2d2 Compare October 29, 2019 04:01
@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
@pertsenga pertsenga force-pushed the laurent-fix-universal-pr-adopted branch from 3476a67 to 8017716 Compare October 31, 2019 04:51
tsconfig.json Outdated Show resolved Hide resolved
@ghost
Copy link

ghost commented Nov 5, 2019

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

@pertsenga pertsenga force-pushed the laurent-fix-universal-pr-adopted branch from 8017716 to 255841f Compare November 6, 2019 02:29
@pertsenga pertsenga force-pushed the laurent-fix-universal-pr-adopted branch from 255841f to 0d8355f Compare November 6, 2019 02:56
@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 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.

None yet

4 participants