Skip to content

Fix Angular SPA Template compiler bug #6308

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

Closed
wants to merge 1 commit into from
Closed

Fix Angular SPA Template compiler bug #6308

wants to merge 1 commit into from

Conversation

speige
Copy link
Contributor

@speige speige commented Jan 2, 2019

@SteveSandersonMS

Summary of the changes (Less than 80 chars)

  • Make StreamReader send last line's contents to listener (otherwise AngularCliBuilder doesn't know when build:ssr is complete)

Addresses #6306

… otherwise AngularCliBuilder doesn't know when build:ssr is complete
@Eilon Eilon added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Jan 3, 2019
@mkArtakMSFT
Copy link
Contributor

@SteveSandersonMS, @ryanbrandenburg can you guys please have a look at this?

@ryanbrandenburg
Copy link
Contributor

I think this one if for @SteveSandersonMS, I don't really have enough context to usefully opine.

@ryanbrandenburg ryanbrandenburg removed their request for review January 29, 2019 21:39
@SteveSandersonMS
Copy link
Member

Looks good - thanks @speige!

@SteveSandersonMS
Copy link
Member

/AzurePipelines run AspNetCore-ci

@SteveSandersonMS
Copy link
Member

/AzurePipelines run all

@SteveSandersonMS
Copy link
Member

Well, I guess that's not the way to trigger a build any more. Or it's just ignoring me for some reason.

@SteveSandersonMS
Copy link
Member

/AzurePipelines run all

@mkArtakMSFT
Copy link
Contributor

/azp run all

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mkArtakMSFT
Copy link
Contributor

@speige can you please rebase this on master and redo it. Sorry, it took us a while to get to this.

@mkArtakMSFT
Copy link
Contributor

Closing this as there was no response from the owner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants