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

Change "current pull request" to "checked out pull request" #1722

Merged
merged 4 commits into from Oct 4, 2018

Conversation

Projects
None yet
3 participants
@masquel
Copy link

masquel commented Oct 4, 2018

Description of the Change

Changed "current pull request" to "checked out pull request" at docs/rfcs/002-issueish-list.md and graphql/schema.graphql files

Alternate Designs

N/A

Benefits

N/A

Possible Drawbacks

N/A

Applicable Issues

#1719

Metrics

N/A

Tests

N/A

Documentation

N/A

@coveralls

This comment has been minimized.

Copy link

coveralls commented Oct 4, 2018

Coverage Status

Coverage remained the same at 81.956% when pulling cefe40c on masquel:master into 3d99d94 on atom:master.

@@ -5284,7 +5284,7 @@ type PullRequestReview implements Node & Comment & Deletable & Updatable & Updat
"""The body of this review rendered as plain text."""
bodyText: String!
"""A list of review comments for the current pull request review."""
"""A list of review comments for the checked out pull request review."""

This comment has been minimized.

Copy link
@smashwilson

smashwilson Oct 4, 2018

Member

This is a generated file, so we shouldn't edit it directly 🙈

This comment has been minimized.

Copy link
@masquel

masquel Oct 4, 2018

Author

ok...

Alexey Alexey

@smashwilson smashwilson merged commit cdce00a into atom:master Oct 4, 2018

6 checks passed

ci/circleci: beta Your tests passed on CircleCI!
Details
ci/circleci: dev Your tests passed on CircleCI!
Details
ci/circleci: stable Your tests passed on CircleCI!
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 81.956%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.