Jesse Rosenberger

Try to use the GraphQLError class to produce the error

3

Just to test this approach

To ensure there aren't any `package-lock.json` changes that are going to crop up. Maybe we can just check if `git status --porcelain` is empty but if we want something more...

originally brought to our attention by @EverlastingBugstopper: > right now we're just calling `.toString()` on the `hints` from the new `composition-js` module, but there's actually more info there. we should...

chore(tests): Apply nextest retries rules to known-flaky tests.

2

### Motivation We have some known flaky tests that are in our test suite. These flaky tests are valuable, but often fail because of network-related environment things or approaches that...

Docs: Update Dynatrace Metrics documentation

2

This ensures that `temporality: delta` is included in the suggestions for Dynatrace.

docs: Add missing slash to containerization overview

1

Just a small thing I saw.

"429 Too Many Requests" not captured in `apollo_router_http_requests_total` metric

1

When `429 Too Many Request`'s are triggered on account of limiting, they aren't counted in the `apollo_router_http_request_total` metric. Unsure if that is intentional or not, but it would be desirable...

`::` is just wrong.

Helm: Use the supergraph file provided

8

I saw @lambertjosh stumble on this at one point and I wrote this quick fix but never opened the PR. I'm not sure what the impact of using a single...

Revert "coprocessor: fix parsing of graphql responses with null as `data` (backport #7141)"

1

Reverts apollographql/router#7315