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.

Sign up for GitHub

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

Jump to bottom

execute: Rename resolveField function and update comments #3159

Merged
merged 3 commits into from
Jun 8, 2021
Merged

execute: Rename resolveField function and update comments #3159

merged 3 commits into from
Jun 8, 2021

Commits on Jun 4, 2021

  1. rename resolveField function 

    to match spec
    @yaacovCR
    yaacovCR committed Jun 4, 2021
    Configuration menu
    Copy the full SHA
    9987cc9 View commit details
    Browse the repository at this point in the history

  2. Latest draft of spec has sections titled "Executing..." not "Evaluati… 

    …ng..."
    @yaacovCR
    yaacovCR committed Jun 4, 2021
    Configuration menu
    Copy the full SHA
    d236611 View commit details
    Browse the repository at this point in the history

  3. remove references to read/write modes 

    spec does not have read/write modes. although write is a reference to
mutations, seems proper to not introduce new concepts within the code
    @yaacovCR
    yaacovCR committed Jun 4, 2021
    Configuration menu
    Copy the full SHA
    480dca9 View commit details
    Browse the repository at this point in the history