Skip to content

test: fix several bazel compiler-cli tests in windows - #30189

Closed
alan-agius4 wants to merge 3 commits into
angular:masterfrom
alan-agius4:windows-compiler-cli-fixes
Closed

alan-agius4 wants to merge 3 commits into
angular:masterfrom
alan-agius4:windows-compiler-cli-fixes

Conversation

@alan-agius4

@alan-agius4 alan-agius4 commented Apr 29, 2019 •

Copy link
Copy Markdown
Contributor
//packages/compiler-cli/integrationtest:integrationtest
//packages/compiler-cli/test/compliance:compliance

Partially addresses #29785

@alan-agius4
alan-agius4 requested a review from a team April 29, 2019 13:44
@alan-agius4 alan-agius4 added area: core Issues related to the framework runtime hotlist: windows-support target: patch This PR is targeted for the next patch release and removed cla: yes labels Apr 29, 2019
@ngbot ngbot Bot added this to the needsTriage milestone Apr 29, 2019
@googlebot

Copy link
Copy Markdown

☹️ Sorry, but only Googlers may change the label cla: yes.

@alan-agius4 alan-agius4 added area: build & ci Related the build and CI infrastructure of the project state: WIP and removed state: WIP labels Apr 29, 2019
@alan-agius4
alan-agius4 requested review from a team April 30, 2019 06:19
```
//packages/compiler-cli/integrationtest:integrationtest
//packages/compiler-cli/test/compliance:compliance
```

Partially addresses #29785
@petebacondarwin
petebacondarwin removed the request for review from a team April 30, 2019 08:21

@devversion devversion left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM for dev-infra.

Also the changes in general look good to me 😄 I did similar stuff in 40d64b6 to get it work on windows.

Comment thread packages/compiler/test/aot/test_util.ts Resolved
Comment thread packages/compiler/test/aot/test_util.ts Resolved
@alxhub alxhub added the action: merge The PR is ready for merge by the caretaker label May 7, 2019
kara pushed a commit that referenced this pull request May 7, 2019
```
//packages/compiler-cli/integrationtest:integrationtest
//packages/compiler-cli/test/compliance:compliance
```

Partially addresses #29785

PR Close #30189
@kara kara closed this in 1660b34 May 7, 2019
@alan-agius4
alan-agius4 deleted the windows-compiler-cli-fixes branch May 7, 2019 17:31
BioPhoton pushed a commit to BioPhoton/angular that referenced this pull request May 21, 2019
```
//packages/compiler-cli/integrationtest:integrationtest
//packages/compiler-cli/test/compliance:compliance
```

Partially addresses angular#29785

PR Close angular#30189
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project area: core Issues related to the framework runtime cla: yes target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants