Skip to content

test: fix ngc compiler tests under windows - #30146

Closed
alan-agius4 wants to merge 2 commits into
angular:masterfrom
alan-agius4:windows-compiler-test
Closed

alan-agius4 wants to merge 2 commits into
angular:masterfrom
alan-agius4:windows-compiler-test

Conversation

@alan-agius4

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

Copy link
Copy Markdown
Contributor

test: fix several Bazel compiler tests in windows

//packages/compiler-cli/test:ngc
//packages/compiler/test:test
//packages/compiler-cli/test/ngtsc:ngtsc

This also address node_modules to the ignored paths for ngc compiler as otherwise the ready is never fired

Partially addresses #29785

@alan-agius4 alan-agius4 added area: build & ci Related the build and CI infrastructure of the project area: core Issues related to the framework runtime hotlist: windows-support target: major This PR is targeted for the next major release labels Apr 26, 2019
@ngbot ngbot Bot modified the milestone: needsTriage Apr 26, 2019
@alan-agius4
alan-agius4 marked this pull request as ready for review April 26, 2019 13:29
@alan-agius4
alan-agius4 requested review from a team April 26, 2019 13:29
@alan-agius4 alan-agius4 added target: patch This PR is targeted for the next patch release and removed target: major This PR is targeted for the next major release labels Apr 26, 2019
@alan-agius4 alan-agius4 added merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note action: presubmit The PR is in need of a google3 presubmit labels May 7, 2019
@ngbot ngbot Bot added the action: merge The PR is ready for merge by the caretaker label May 7, 2019
@ngbot

This comment has been minimized.

@alan-agius4 alan-agius4 removed merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note action: merge The PR is ready for merge by the caretaker labels May 7, 2019
@alxhub

alxhub commented May 7, 2019

Copy link
Copy Markdown
Member

Presubmit

```
//packages/compiler-cli/test:ngc
//packages/compiler/test:test
```

This also address `node_modules` to the ignored paths for ngc compiler as otherwise the `ready` is never fired

Partially addresses #29785
This commit fixes the following test target in windows

```
//packages/compiler-cli/test/ngtsc:ngtsc
```
@alexeagle

Copy link
Copy Markdown
Contributor

presubmit

@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note and removed action: presubmit The PR is in need of a google3 presubmit labels May 13, 2019
alxhub pushed a commit that referenced this pull request May 13, 2019
```
//packages/compiler-cli/test:ngc
//packages/compiler/test:test
```

This also address `node_modules` to the ignored paths for ngc compiler as otherwise the `ready` is never fired

Partially addresses #29785

PR Close #30146
alxhub pushed a commit that referenced this pull request May 13, 2019
This commit fixes the following test target in windows

```
//packages/compiler-cli/test/ngtsc:ngtsc
```

PR Close #30146
@alxhub alxhub closed this in 31df513 May 13, 2019
alxhub pushed a commit that referenced this pull request May 13, 2019
This commit fixes the following test target in windows

```
//packages/compiler-cli/test/ngtsc:ngtsc
```

PR Close #30146
@alan-agius4
alan-agius4 deleted the windows-compiler-test branch May 13, 2019 18:17
BioPhoton pushed a commit to BioPhoton/angular that referenced this pull request May 21, 2019
```
//packages/compiler-cli/test:ngc
//packages/compiler/test:test
```

This also address `node_modules` to the ignored paths for ngc compiler as otherwise the `ready` is never fired

Partially addresses angular#29785

PR Close angular#30146
BioPhoton pushed a commit to BioPhoton/angular that referenced this pull request May 21, 2019
This commit fixes the following test target in windows

```
//packages/compiler-cli/test/ngtsc:ngtsc
```

PR Close angular#30146
@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 merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note 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