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

[Merged by Bors] - Fix the scalar derive. #401

Closed
wants to merge 1 commit into from
Closed

Conversation

obmarg
Copy link
Owner

@obmarg obmarg commented Feb 20, 2022

Why are we making this change?

#391 changed the way scalar derives worked, but it didn't handle generic types inside a scalar derive.

What effects does this change have?

Fixes generic types in a scalar derive, adds a test.

@obmarg
Copy link
Owner Author

obmarg commented Feb 20, 2022

bors r+

@netlify
Copy link

netlify bot commented Feb 20, 2022

✔️ Deploy Preview for cynic-querygen-web canceled.

🔨 Explore the source changes: 47f3706

🔍 Inspect the deploy log: https://app.netlify.com/sites/cynic-querygen-web/deploys/62129cee32886900073325e8

bors bot pushed a commit that referenced this pull request Feb 20, 2022
#### Why are we making this change?

#391 changed the way scalar derives worked, but it didn't handle generic types inside a scalar derive.

#### What effects does this change have?

Fixes generic types in a scalar derive, adds a test.
@bors
Copy link

bors bot commented Feb 20, 2022

Canceled.

@obmarg
Copy link
Owner Author

obmarg commented Feb 20, 2022

bors r+

bors bot pushed a commit that referenced this pull request Feb 20, 2022
#### Why are we making this change?

#391 changed the way scalar derives worked, but it didn't handle generic types inside a scalar derive.

#### What effects does this change have?

Fixes generic types in a scalar derive, adds a test.
@bors
Copy link

bors bot commented Feb 20, 2022

Build failed:

@obmarg
Copy link
Owner Author

obmarg commented Feb 20, 2022

bors r+

bors bot pushed a commit that referenced this pull request Feb 20, 2022
#### Why are we making this change?

#391 changed the way scalar derives worked, but it didn't handle generic types inside a scalar derive.

#### What effects does this change have?

Fixes generic types in a scalar derive, adds a test.
@bors
Copy link

bors bot commented Feb 20, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title Fix the scalar derive. [Merged by Bors] - Fix the scalar derive. Feb 20, 2022
@bors bors bot closed this Feb 20, 2022
@obmarg obmarg deleted the fix-scalar-derive branch May 31, 2023 21:11
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.

1 participant