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

switches gatsby theme #5

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

switches gatsby theme #5

wants to merge 7 commits into from

Conversation

ksmacleod99
Copy link
Collaborator

uses advanced starter for more flexibility. Cypress should still be in there. Pretty sure I got emotion and theme-ui in there as well.
It's ugly, but that means more potential for contribution!

Copy link
Member

@kylemh kylemh left a comment

Choose a reason for hiding this comment

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

There's a lot going on here, but I'm okay with merging it after these changes.

After we merge the PR, I want to get rid of the stylesheets and instead use Emotion to style things in here. We'll also wanna hide and rotate keys for different tools/API access.

Comment on lines -5 to -6
yarn-debug.log*
yarn-error.log*
Copy link
Member

Choose a reason for hiding this comment

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

we should keep these

@@ -1,5 +0,0 @@
{
Copy link
Member

Choose a reason for hiding this comment

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

we should keep this. It's the config for the local git hooks.

@@ -1,3 +0,0 @@
{
Copy link
Member

Choose a reason for hiding this comment

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

we should keep this. It's how we defined what files need to be formatted per commit

@@ -1,3 +0,0 @@
trailingComma: "all"
Copy link
Member

Choose a reason for hiding this comment

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

we should keep this. It will eventually match front-end

@@ -0,0 +1,148 @@
code[class*="language-"],
Copy link
Member

Choose a reason for hiding this comment

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

what's with this theme?

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