Skip to content

Conversation

martinbonnin
Copy link
Contributor

Some users are doing this:

type Query {
  foo: Int @deprecated(reason: "")
}

The motivation is questionable but nevertheless, empty != absent and we should be consistent here.

@martinbonnin martinbonnin requested a review from BoD as a code owner October 3, 2025 09:42
@apollo-librarian
Copy link

apollo-librarian bot commented Oct 3, 2025

✅ Docs preview has no changes

The preview was not built because there were no changes.

Build ID: 16a031123ce21de452137eef
Build Logs: View logs

@martinbonnin martinbonnin changed the title [compiler] Allow empty descriptions and/or deprecationReasons [compiler] Allow empty deprecationReasons Oct 3, 2025
Copy link
Contributor

@BoD BoD left a comment

Choose a reason for hiding this comment

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

👍

@martinbonnin martinbonnin merged commit 1d12ad6 into main Oct 6, 2025
10 of 11 checks passed
@martinbonnin martinbonnin deleted the empty-deprecations branch October 6, 2025 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants