Skip to content

Comments

reorganize gatsby related file structure#35

Open
interim17 wants to merge 20 commits intomainfrom
feature/gatsby-org
Open

reorganize gatsby related file structure#35
interim17 wants to merge 20 commits intomainfrom
feature/gatsby-org

Conversation

@interim17
Copy link
Contributor

Problem

Housekeeping. Moving things into this structure:

/gatsby
   constants.js
   /schema
      .gql files
  /utils
     .resolvers etc.

I tried moving gastby-node and gatsby-config in there and exporting those files from the root but it seemed annoying.
Gatsby expects those at the project root so I just left them there for now,

@interim17 interim17 marked this pull request as ready for review January 23, 2026 23:14
@interim17 interim17 requested a review from Copilot January 23, 2026 23:15
Copy link
Contributor

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 PR reorganizes the Gatsby-related file structure by consolidating files into a clearer directory hierarchy under /gatsby, with subdirectories for /utils, /schema, and a constants.js file. The primary purpose is housekeeping to improve code organization and maintainability.

Changes:

  • Moved constants from gatsbyutils/constants to gatsby/constants
  • Moved resolver utilities from gatsbyutils/ to gatsby/utils/
  • Updated import paths throughout the codebase to reflect the new structure

Reviewed changes

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

File Description
gatsby/utils/test/gatsby-resolver-utils.test.js Updated import path for constants to match new directory structure
gatsby/utils/gatsby-resolver-utils.js Updated require path for constants to match new directory structure
gatsby-node.js Updated require paths for resolver utils and constants to match new directory structure

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@meganrm
Copy link
Collaborator

meganrm commented Feb 20, 2026

I'm going to wait until you merge the one this is branched from to look at this

Base automatically changed from feature/gql to main February 20, 2026 21:55
@interim17
Copy link
Contributor Author

@meganrm I merged the other branch!

@netlify
Copy link

netlify bot commented Feb 23, 2026

Deploy Preview for project-idea-board ready!

Name Link
🔨 Latest commit f211c7d
🔍 Latest deploy log https://app.netlify.com/projects/project-idea-board/deploys/699cc5d2e02cab0008548984
😎 Deploy Preview https://deploy-preview-35--project-idea-board.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

2 participants