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

Moved GraphQL TypeScript to __graphql__ #588

Conversation

CatChen
Copy link
Owner

@CatChen CatChen commented Aug 2, 2024

No description provided.

Copy link
Owner Author

CatChen commented Aug 2, 2024

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CatChen and the rest of your teammates on Graphite Graphite

Copy link

what-the-diff bot commented Aug 2, 2024

PR Summary

  • .npmignore updated
    The .npmignore file was updated to ensure that the codegen.ts and schema.graphql files are excluded from the npm package.

  • Files renamed and relocated in source bundle
    Renamed files in the source bundle's GraphQL directory to a new directory named __graphql__.

  • Type generation updates
    The codegen.ts file has been adjusted for generating types and operations in the newly renamed __graphql__ directory.

  • ESLint configuration modification
    The eslint.config.js file was updated to exclude the new __graphql__ directory from linting.

  • Package.json codegen script updated
    The codegen script in package.json now removes the __graphql__ directory before generating new code to ensure that old files do not remain.

  • Renamed files in the source directory
    The GraphQL files in the source directory were renamed and moved to the __graphql__.

  • Source Index file updated
    The src/index.ts file was modified to import gql.js from the new __graphql__ directory, reflecting the file restructures and renaming.

@CatChen CatChen force-pushed the 08-01-set_up_graphql_typescript_client branch from eda5ce6 to 704f8a0 Compare August 2, 2024 21:43
@CatChen CatChen force-pushed the 08-02-moved_graphql_typescript_to___graphql__ branch from 7b66d76 to 8654f82 Compare August 2, 2024 21:43
@CatChen CatChen force-pushed the 08-01-set_up_graphql_typescript_client branch from 704f8a0 to fc2033c Compare August 2, 2024 21:45
@CatChen CatChen force-pushed the 08-02-moved_graphql_typescript_to___graphql__ branch from 8654f82 to 9e6ed27 Compare August 2, 2024 21:45
@CatChen CatChen self-assigned this Aug 2, 2024
@CatChen CatChen marked this pull request as ready for review August 2, 2024 21:48
@CatChen CatChen force-pushed the 08-01-set_up_graphql_typescript_client branch from 8841980 to 63dd26c Compare August 2, 2024 21:50
@CatChen CatChen force-pushed the 08-02-moved_graphql_typescript_to___graphql__ branch 4 times, most recently from c8d87df to af4b82a Compare August 4, 2024 04:39
@CatChen CatChen force-pushed the 08-02-moved_graphql_typescript_to___graphql__ branch from af4b82a to e5f8a61 Compare August 4, 2024 04:41
@CatChen CatChen merged commit 94d8c14 into 08-01-set_up_graphql_typescript_client Aug 4, 2024
12 checks passed
@CatChen CatChen deleted the 08-02-moved_graphql_typescript_to___graphql__ branch August 4, 2024 04:47
CatChen added a commit that referenced this pull request Aug 4, 2024
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.

1 participant