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

Use ID! type instead of String! #375

Merged
merged 1 commit into from
Feb 28, 2022
Merged

Conversation

CasperWA
Copy link
Contributor

Description:

Use type ID! instead of String! in the auto-merge workflow for a GitHub GraphQL query.

This PR can be forced through as a proper review makes no sense.
Furthermore, whether it works or not will be made clear in #373.

Fixes #374

Type of change:

  • Bug fix.
  • New feature.
  • Documentation update.

Checklist:

This checklist can be used as a help for the reviewer.

  • Is the code easy to read and understand?
  • Are comments for humans to read, not computers to disregard?
  • Does a new feature has an accompanying new test (in the CI or unit testing schemes)?
  • Has the documentation been updated as necessary?
  • Does this close the issue?
  • Is the change limited to the issue?
  • Are errors handled for all outcomes?
  • Does the new feature provide new restrictions on dependencies, and if so is this documented?

Comments:

@CasperWA CasperWA merged commit e36eab6 into master Feb 28, 2022
@CasperWA CasperWA deleted the cwa/fix-374-gh-graphql-type-fix branch February 28, 2022 14:05
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.

GH GraphQL type issue for auto-merge workflow
1 participant