Skip to content

🐛 fix merging graphql-config with inline config#2362

Merged
edno merged 3 commits intomainfrom
fix/option-merge-gqlconfig
Aug 27, 2025
Merged

🐛 fix merging graphql-config with inline config#2362
edno merged 3 commits intomainfrom
fix/option-merge-gqlconfig

Conversation

@edno
Copy link
Copy Markdown
Member

@edno edno commented Aug 27, 2025

Description

Fix issue when graphql-config config properties are not correctly merged with inline config (eg docOptions.frontMatter). This PR replace the ... merge by @fastify/deepmerge to ensure that all properties are correctly merged.

Checklist

  • My changes follow the contributing guidelines of this project.
  • I have performed a self-review of my code.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my changes work.
  • New and existing unit tests pass locally with my changes.

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@edno edno merged commit 84d29e9 into main Aug 27, 2025
32 checks passed
@edno edno deleted the fix/option-merge-gqlconfig branch August 27, 2025 19:25
@edno edno mentioned this pull request Aug 27, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant