test(docs-infra): run the orphaned shared-docs pipes specs by erkamyaman · Pull Request #70585 · angular/angular · GitHub
Skip to content

test(docs-infra): run the orphaned shared-docs pipes specs - #70585

Open
erkamyaman wants to merge 1 commit into
angular:mainfrom
erkamyaman:run-shared-docs-pipes-specs
Open

test(docs-infra): run the orphaned shared-docs pipes specs#70585
erkamyaman wants to merge 1 commit into
angular:mainfrom
erkamyaman:run-shared-docs-pipes-specs

Conversation

@erkamyaman

Copy link
Copy Markdown
Contributor

relative-link.pipe.spec.ts and is-active-navigation-item.pipe.spec.ts arrived with this package's BUILD file in #57132, but no test target came with them and lib excludes **/*.spec.ts, so nothing has ever compiled them. bazel query reports both as not declared in the package.

They pass unmodified. getRelativeUrl has no other coverage in the repo, and both pipes are used by the search dialog, the navigation list and the not found page.

`relative-link.pipe.spec.ts` and `is-active-navigation-item.pipe.spec.ts`
arrived with this package's BUILD file in angular#57132, but no test target came with
them and `lib` excludes `**/*.spec.ts`, so nothing has ever compiled them.
`bazel query` reports both as not declared in the package.

They pass unmodified. `getRelativeUrl` has no other coverage in the repo, and
both pipes are used by the search dialog, the navigation list and the not
found page.
@pullapprove
pullapprove Bot requested a review from atscott September 5, 2026 21:41
@angular-robot angular-robot Bot added the area: docs-infra Angular.dev application and infrastructure label Sep 5, 2026
@ngbot ngbot Bot added this to the Backlog milestone Sep 5, 2026
@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Sep 5, 2026
@JeanMeche
JeanMeche removed the request for review from atscott September 5, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: docs-infra Angular.dev application and infrastructure target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants