Skip to content

Updating angular 6 and fixing webpack issues and rxjs issue. #2007

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

Conversation

govi2010
Copy link

@govi2010 govi2010 commented May 5, 2018

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Bug Fix and Angular 6 update

  • What is the current behavior? (You can also link to an open issue here)
    Currently its working angular 5 with web-pack 3

  • What is the new behavior (if this is a feature change)?
    Now its working angular 6 with web-pack 4

@PatrickJS
Copy link
Owner

can you fix the ci which looks like a problem running the webpack bundle for tests

@PatrickJS PatrickJS changed the title Updateing angular 6 and ficing webpack issues and rxjs issue. Updating angular 6 and fixing webpack issues and rxjs issue. May 5, 2018
@govi2010
Copy link
Author

govi2010 commented May 5, 2018 via email

@govi2010
Copy link
Author

govi2010 commented May 9, 2018

@gdi2290 I have fixed ci issue.

@PatrickJS PatrickJS merged commit 028533b into PatrickJS:feature/migrate-to-webpack4 May 9, 2018
@klihelp klihelp mentioned this pull request May 10, 2018
32 tasks
@govi2010 govi2010 mentioned this pull request May 18, 2018
@MartinMa
Copy link

Works fine here! 🎉

Good to merge in my opinion.

Windows 10
npm 6.0.1
node v8.11.2

As a sidenode. npm complains about a few security vulnerabilities found in the dev dependencies. But I assume most of them can't be fixed right away, because they are third-party dependencies.

[!] 86 vulnerabilities found [30118 packages audited]
    Severity: 5 Low | 75 Moderate | 5 High | 1 Critical

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.

3 participants