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

vscode settings and reccomendations #1

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

acao
Copy link
Contributor

@acao acao commented Nov 16, 2020

default vscode settings for the workspace and plugin reccomendations! check it out. vscode-graphql features will appear after develop process has been run. probably should add docs for that, maybe a launcher as wel??

@TylerBarnes
Copy link
Collaborator

Awesome!

@acao
Copy link
Contributor Author

acao commented Nov 17, 2020

also it looks like we are adding a yarn.lock here, is that customary for a starter?

@TylerBarnes
Copy link
Collaborator

Yeah, I think yarn.lock should be added. Although maybe package-lock.json is standard 🤔 not entirely sure. Maybe we should leave it out though since this will be a git sub-module.

Though I'm wondering if we should maybe wait to merge this PR until the other features for the GraphQL extension are working again. What do you think? The starter should ideally be a good starting point for someone brand new to Gatsby so if we add the GraphQL plugin and folks can't quite yet understand/see what it's for it might be confusing.

@acao acao force-pushed the chore/vscode-config branch from 75821ff to 0ddaad9 Compare November 17, 2020 01:01
@acao
Copy link
Contributor Author

acao commented Nov 17, 2020

@TylerBarnes i think gatsby is standardizing around npm, so i've re-written the last commit to add that instead. adding the lockfile might just make sense in a different PR though.

it adds a lot of functionality as it is, and works out of the box, and it will be a long while before I'll be able to find the time to fix those (non visible) bugs in the LSP

@@ -37,6 +37,7 @@ module.exports = {
*/
`gatsby-transformer-sharp`,
`gatsby-plugin-sharp`,
`gatsby-plugin-graphql-config`,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you add some space above this + a comment linking to the GraphQL config plugin docs? Similar to the style of the other plugins. I want to make sure any new folks who use this can "explore or ignore" 😛

@TylerBarnes
Copy link
Collaborator

Sounds good! In that case I left 1 comment :)

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