Skip to content

Comments

refactor(execution): add separate entrypoints file#4527

Merged
yaacovCR merged 1 commit intographql:nextfrom
yaacovCR:entrypoints
Jan 26, 2026
Merged

refactor(execution): add separate entrypoints file#4527
yaacovCR merged 1 commit intographql:nextfrom
yaacovCR:entrypoints

Conversation

@yaacovCR
Copy link
Contributor

@yaacovCR yaacovCR commented Jan 26, 2026

cleans up our long execute file (a bit)

it's potentially a breaking change if these entrypoints were being imported directly; we will clean that up later by renaming execute.ts to something else and renaming entrypoints.ts to execute.ts, but removing the entrypoints first will hopefully allow us to preserve more history.

UPDATE: #4539 renames it back, finishing up the effort to break up the file without losing so much of the history, no longer a breaking change.

@yaacovCR yaacovCR requested a review from a team as a code owner January 26, 2026 11:09
@yaacovCR yaacovCR added PR: breaking change 💥 implementation requires increase of "major" version number PR: polish 💅 PR doesn't change public API or any observed behaviour labels Jan 26, 2026
@vercel
Copy link

vercel bot commented Jan 26, 2026

@yaacovCR is attempting to deploy a commit to the The GraphQL Foundation Team on Vercel.

A member of the Team first needs to authorize it.

@yaacovCR yaacovCR merged commit a37ce63 into graphql:next Jan 26, 2026
15 of 16 checks passed
@yaacovCR yaacovCR deleted the entrypoints branch January 26, 2026 11:15
yaacovCR added a commit to yaacovCR/graphql-js that referenced this pull request Jan 27, 2026
cleans up our long execute file (a bit)

it's potentially a breaking change if these entrypoints were being
imported directly; we will clean that up later by renaming execute.ts to
something else and renaming entrypoints.ts to execute.ts, but removing
the entrypoints first will hopefully allow us to preserve more history.
yaacovCR added a commit that referenced this pull request Jan 28, 2026
in prelude to renaming `entrypoints.ts` back to `execute.ts` finishing
off #4527
@yaacovCR yaacovCR removed the PR: breaking change 💥 implementation requires increase of "major" version number label Jan 28, 2026
yaacovCR added a commit that referenced this pull request Jan 28, 2026
yaacovCR added a commit that referenced this pull request Feb 22, 2026
cleans up our long execute file (a bit)

it's potentially a breaking change if these entrypoints were being
imported directly; we will clean that up later by renaming execute.ts to
something else and renaming entrypoints.ts to execute.ts, but removing
the entrypoints first will hopefully allow us to preserve more history.
yaacovCR added a commit that referenced this pull request Feb 22, 2026
in prelude to renaming `entrypoints.ts` back to `execute.ts` finishing
off #4527
yaacovCR added a commit that referenced this pull request Feb 22, 2026
yaacovCR added a commit that referenced this pull request Feb 22, 2026
cleans up our long execute file (a bit)

it's potentially a breaking change if these entrypoints were being
imported directly; we will clean that up later by renaming execute.ts to
something else and renaming entrypoints.ts to execute.ts, but removing
the entrypoints first will hopefully allow us to preserve more history.
yaacovCR added a commit that referenced this pull request Feb 22, 2026
in prelude to renaming `entrypoints.ts` back to `execute.ts` finishing
off #4527
yaacovCR added a commit that referenced this pull request Feb 22, 2026
yaacovCR added a commit that referenced this pull request Feb 23, 2026
cleans up our long execute file (a bit)

it's potentially a breaking change if these entrypoints were being
imported directly; we will clean that up later by renaming execute.ts to
something else and renaming entrypoints.ts to execute.ts, but removing
the entrypoints first will hopefully allow us to preserve more history.
yaacovCR added a commit that referenced this pull request Feb 23, 2026
in prelude to renaming `entrypoints.ts` back to `execute.ts` finishing
off #4527
yaacovCR added a commit that referenced this pull request Feb 23, 2026
yaacovCR added a commit that referenced this pull request Feb 23, 2026
in prelude to renaming `entrypoints.ts` back to `execute.ts` finishing
off #4527
yaacovCR added a commit that referenced this pull request Feb 23, 2026
yaacovCR added a commit that referenced this pull request Feb 24, 2026
cleans up our long execute file (a bit)

it's potentially a breaking change if these entrypoints were being
imported directly; we will clean that up later by renaming execute.ts to
something else and renaming entrypoints.ts to execute.ts, but removing
the entrypoints first will hopefully allow us to preserve more history.
yaacovCR added a commit that referenced this pull request Feb 24, 2026
in prelude to renaming `entrypoints.ts` back to `execute.ts` finishing
off #4527
yaacovCR added a commit that referenced this pull request Feb 24, 2026
yaacovCR added a commit that referenced this pull request Feb 24, 2026
cleans up our long execute file (a bit)

it's potentially a breaking change if these entrypoints were being
imported directly; we will clean that up later by renaming execute.ts to
something else and renaming entrypoints.ts to execute.ts, but removing
the entrypoints first will hopefully allow us to preserve more history.
yaacovCR added a commit that referenced this pull request Feb 24, 2026
in prelude to renaming `entrypoints.ts` back to `execute.ts` finishing
off #4527
yaacovCR added a commit that referenced this pull request Feb 24, 2026
yaacovCR added a commit that referenced this pull request Feb 24, 2026
cleans up our long execute file (a bit)

it's potentially a breaking change if these entrypoints were being
imported directly; we will clean that up later by renaming execute.ts to
something else and renaming entrypoints.ts to execute.ts, but removing
the entrypoints first will hopefully allow us to preserve more history.
yaacovCR added a commit that referenced this pull request Feb 24, 2026
in prelude to renaming `entrypoints.ts` back to `execute.ts` finishing
off #4527
yaacovCR added a commit that referenced this pull request Feb 24, 2026
yaacovCR added a commit that referenced this pull request Feb 24, 2026
cleans up our long execute file (a bit)

it's potentially a breaking change if these entrypoints were being
imported directly; we will clean that up later by renaming execute.ts to
something else and renaming entrypoints.ts to execute.ts, but removing
the entrypoints first will hopefully allow us to preserve more history.
yaacovCR added a commit that referenced this pull request Feb 24, 2026
in prelude to renaming `entrypoints.ts` back to `execute.ts` finishing
off #4527
yaacovCR added a commit that referenced this pull request Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: polish 💅 PR doesn't change public API or any observed behaviour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant