Skip to content

Repo sync #37948

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
merged 3 commits into from
Apr 30, 2025
Merged

Repo sync #37948

merged 3 commits into from
Apr 30, 2025

Conversation

docs-bot
Copy link
Collaborator

This is an automated pull request to sync changes between the public and private repos.
Our bot will merge this pull request automatically.
To preserve continuity across repos, do not squash this pull request.

@Copilot Copilot AI review requested due to automatic review settings April 30, 2025 09:34
@github github locked as spam and limited conversation to collaborators Apr 30, 2025
@docs-bot docs-bot merged commit 3a7392f into main Apr 30, 2025
1 check passed
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This automated pull request synchronizes configuration and schema files between the public and private repositories.

  • SHA values have been updated in several config files.
  • Schema descriptions for alert filtering across multiple REST API versions have been updated.
  • GraphQL schema fields for position parameters have been updated to be nullable with modified descriptions.

Reviewed Changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/webhooks/lib/config.json Updated SHA value
src/rest/lib/config.json Updated SHA value
src/rest/data/ghes-3.16-2022-11-28/schema.json Updated alert filter descriptions
src/rest/data/ghes-3.15-2022-11-28/schema.json Updated alert filter descriptions
src/rest/data/ghes-3.14-2022-11-28/schema.json Updated alert filter descriptions
src/rest/data/ghes-3.13-2022-11-28/schema.json Updated alert filter descriptions
src/rest/data/ghes-3.12-2022-11-28/schema.json Updated alert filter descriptions
src/rest/data/ghec-2022-11-28/schema.json Updated alert filter descriptions
src/rest/data/fpt-2022-11-28/schema.json Updated alert filter descriptions
src/graphql/data/ghec/schema.json Changed GraphQL field types for “line” and “path” along with updated descriptions
src/graphql/data/ghec/schema.docs.graphql Updated documentation for GraphQL “line” and “path” fields
src/graphql/data/fpt/schema.json Changed GraphQL field types for “line” and “path” along with updated descriptions
src/graphql/data/fpt/schema.docs.graphql Updated documentation for GraphQL “line” and “path” fields
src/graphql/data/fpt/changelog.json Added changelog entries reflecting GraphQL schema changes
src/github-apps/lib/config.json Updated SHA value
Comments suppressed due to low confidence (4)

src/graphql/data/ghec/schema.json:102379

  • Changing the field type from non-nullable (Int!) to nullable (Int) while the description indicates it is required may confuse API consumers. Consider ensuring consistency between the field type and its description.
"description": "<p>The line of the blob to which the thread refers. The end of the line range for\nmulti-line comments. Required if not using positioning.</p>",

src/graphql/data/ghec/schema.json:102387

  • Changing the field type from non-nullable (String!) to nullable (String) while the description suggests it is required may cause confusion. Review the API contract to ensure clarity for consumers.
"description": "<p>Path to the file being commented on. Required if not using positioning.</p>",

src/graphql/data/fpt/schema.json:102379

  • The change from a non-nullable type (Int!) to a nullable type (Int) for the line field, while its description implies it is required, might lead to integration issues. Validate whether the API should enforce non-nullability or adjust the documentation accordingly.
"description": "<p>The line of the blob to which the thread refers. The end of the line range for\nmulti-line comments. Required if not using positioning.</p>",

src/graphql/data/fpt/schema.json:102387

  • Altering the type for the path field from String! to String, while the updated description marks it as required, may create uncertainty for API users. Consider aligning the type declaration with the stated requirement.
"description": "<p>Path to the file being commented on. Required if not using positioning.</p>",

@github-actions github-actions bot added the triage Do not begin working on this issue until triaged by the team label Apr 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
triage Do not begin working on this issue until triaged by the team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants