Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fusion] Added post-merge validation rule "EmptyMergedInputObjectTypeRule" #7993

Merged
merged 3 commits into from
Feb 5, 2025

Conversation

danielreynolds1
Copy link
Collaborator

@danielreynolds1 danielreynolds1 commented Feb 4, 2025

Summary of the changes (Less than 80 chars)

  • [Fusion] Added post-merge validation rule "EmptyMergedInputObjectTypeRule".

@danielreynolds1 danielreynolds1 force-pushed the dr/empty-merged-input-object-type-rule branch from 1ac70af to b85419f Compare February 4, 2025 21:38
Copy link

codecov bot commented Feb 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.41%. Comparing base (6c393d3) to head (682d464).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7993   +/-   ##
=======================================
  Coverage   74.41%   74.41%           
=======================================
  Files        2898     2898           
  Lines      152294   152294           
  Branches    17779    17779           
=======================================
+ Hits       113323   113327    +4     
+ Misses      33076    33070    -6     
- Partials     5895     5897    +2     
Flag Coverage Δ
unittests 74.41% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@glen-84 glen-84 marked this pull request as ready for review February 5, 2025 11:01
@glen-84 glen-84 changed the title [Fusion] Added post-merge validation rule EmptyMergedInputObjectTypeRule [Fusion] Added post-merge validation rule "EmptyMergedInputObjectTypeRule" Feb 5, 2025
@glen-84 glen-84 merged commit fc66e4e into main Feb 5, 2025
106 checks passed
@glen-84 glen-84 deleted the dr/empty-merged-input-object-type-rule branch February 5, 2025 11:03
michaelstaib added a commit that referenced this pull request Feb 19, 2025
* [Fusion] Added post-merge validation rule "EmptyMergedInputObjectTypeRule" (#7993)

* [Fusion] Refactored merging to ensure correct type references (#7995)

* Fixed CookieCrumble Package Configurations. (#7998)

* [Fusion] Added merging of implemented interfaces (#7999)

* Added more include overloads. (#8002)

* [Fusion] Used `@fusion__inaccessible` in the execution schema (#8005)

* Added support for LocalDate/LocalDateTime/LocalTime to Strawberry Shake (#8007)

* Request totalCount through ToPageAsync if its requested. (#8011)

* Added QueryContext extension methods for Select and Include. (#8013)

* Switched LangVersion from `preview` to `13.0` (#8014)

* Updated .NET 9 packages (#8012)

* Changed the runtime type for the Date scalar in Strawberry Shake (#8009)

* Added Strawberry Shake runtime type change to migration guide (#8010)

* Migrated to new solution file format. (#8027)

* Combining the CompilationProvider is an anti-pattern, create an IncrementalValueProvider for the assembly name instead. (#8021)

* Fixed ignored invalid field when enum is present (#8026)

* Allow for internal DataLoader modules and groups. (#8028)

* Updated the latest stable version for the docs to v15 (#8018)

* Added documentation directory for Fusion v15 (#8017)

* Added Strawberry Shake migration documents to `docs.json` (#8015)

* [Fusion] Fix explosion of selections (#8030)

---------

Co-authored-by: Daniel Reynolds <[email protected]>
Co-authored-by: Michael Staib <[email protected]>
Co-authored-by: Martin Gasser <[email protected]>
Co-authored-by: faddiv <[email protected]>
Co-authored-by: Tobias Tengler <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants