Skip to content

Revert "run e2e legacy tests under bazel" - #24335

Merged
angular-robot[bot] merged 4 commits into
mainfrom
revert-23074-legacy-test
Nov 28, 2022
Merged

Revert "run e2e legacy tests under bazel"#24335
angular-robot[bot] merged 4 commits into
mainfrom
revert-23074-legacy-test

Conversation

@alan-agius4

@alan-agius4 alan-agius4 commented Nov 28, 2022

Copy link
Copy Markdown
Collaborator

Reverts #23074

Reverting as this is failing on "main".

https://app.circleci.com/pipelines/github/angular/angular-cli/28109/workflows/3b36eeae-67ea-4ce4-a3a9-a82adb399f30
https://app.circleci.com/pipelines/github/angular/angular-cli/28106/workflows/a8dec6ca-16b1-4781-a5db-58a01fe39b97

Example of errors

STDERR:
npm ERR! code 1
npm ERR! The git reference could not be found
npm ERR! command git --no-replace-objects checkout 9e5777052
npm ERR! error: pathspec '9e5777052' did not match any file(s) known to git

npm ERR! A complete log of this run can be found in:
npm ERR!     /mnt/ramdisk/e2e/angular-cli-e2e-9qbv9h/npm-cache/_logs/2022-11-28T13_54_25_607Z-debug-0.log

//cc @jbedard & @josephperrott

@alan-agius4
alan-agius4 requested a review from clydin November 28, 2022 14:06
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release labels Nov 28, 2022
@angular-robot
angular-robot Bot merged commit b6f4b54 into main Nov 28, 2022
@alan-agius4
alan-agius4 deleted the revert-23074-legacy-test branch November 29, 2022 15:05
@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 Dec 30, 2022
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 target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants