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

fix: duplicated gatsby config #61

Merged
merged 1 commit into from
Nov 15, 2019
Merged

fix: duplicated gatsby config #61

merged 1 commit into from
Nov 15, 2019

Conversation

afc163
Copy link
Contributor

@afc163 afc163 commented Nov 15, 2019

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Description of change

Copy link

@tests-checker tests-checker bot left a comment

Choose a reason for hiding this comment

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

Could you please add tests to make sure this change works as expected?

@afc163 afc163 changed the title chore: fix gatsby config fix: duplicated gatsby config Nov 15, 2019
@auto-add-label auto-add-label bot added bug 这个是一个 bug and removed chore labels Nov 15, 2019
@lzxue lzxue closed this Nov 15, 2019
@lzxue lzxue reopened this Nov 15, 2019
@lzxue lzxue merged commit 1faeda9 into site Nov 15, 2019
@lzxue lzxue deleted the fix-gatsby-config branch November 15, 2019 11:02
@lgtm-com
Copy link

lgtm-com bot commented Nov 15, 2019

This pull request fixes 1 alert when merging 939ec59 into f073b39 - view on LGTM.com

fixed alerts:

  • 1 for Overwritten property

lzxue added a commit that referenced this pull request Mar 5, 2020
lzxue added a commit that referenced this pull request Mar 5, 2020
lzxue added a commit that referenced this pull request Mar 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 这个是一个 bug PR: merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants