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

hi #1879

Merged
merged 100 commits into from
Mar 31, 2022
Merged

hi #1879

merged 100 commits into from
Mar 31, 2022

Conversation

bgoonz
Copy link
Owner

@bgoonz bgoonz commented Mar 31, 2022

No description provided.

@vercel
Copy link

vercel bot commented Mar 31, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/bgoonz/bgoonz-blog-2-0/5tUYxA83m9TixhFzRqsVZHXWTKiU
✅ Preview: In Progress

@bgoonz bgoonz marked this pull request as ready for review March 31, 2022 05:03
@bgoonz bgoonz merged commit efcc4be into preview Mar 31, 2022
@netlify
Copy link

netlify bot commented Mar 31, 2022

Deploy Preview for bgoonz-blog ready!

Name Link
🔨 Latest commit 88faf88
🔍 Latest deploy log https://app.netlify.com/sites/bgoonz-blog/deploys/62453605c667330008192216
😎 Deploy Preview https://deploy-preview-1879--bgoonz-blog.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 site settings.

@viezly
Copy link

viezly bot commented Mar 31, 2022

This pull request is split into 84 parts for easier review.
👀 Review pull request on Viezly

Changed files are located in these folders:

  • /
  • docs/content
  • docs/js
  • notes
  • notes/_SCRAp
  • notes/docs/BGOONZ_BLOG_2.0.wiki
  • notes/future-implementations/src
  • plugins/gatsby-remark-page-creator
  • plugins/gatsby-source-data
  • src/components
  • src/data
  • src/layouts
  • src/pages
  • src/sass/imports
  • src/utils
  • static/admin
  • static/css
  • static/images
  • static/js

@lgtm-com
Copy link

lgtm-com bot commented Mar 31, 2022

This pull request introduces 6 alerts and fixes 256 when merging 88faf88 into 0db26e7 - view on LGTM.com

new alerts:

  • 6 for Syntax error

fixed alerts:

  • 110 for Unused variable, import, function or class
  • 46 for Useless conditional
  • 26 for Useless assignment to local variable
  • 26 for Comparison between inconvertible types
  • 20 for Superfluous trailing arguments
  • 4 for Self assignment
  • 4 for Bad HTML filtering regexp
  • 3 for Implicit operand conversion
  • 3 for Return statement assigns local variable
  • 3 for Unneeded defensive code
  • 2 for Identical operands
  • 2 for Call to eval-like DOM function
  • 2 for Illegal invocation
  • 1 for DOM text reinterpreted as HTML
  • 1 for Prototype-polluting function
  • 1 for Useless assignment to property
  • 1 for Incomplete string escaping or encoding
  • 1 for Invocation of non-function

@ghost
Copy link

ghost commented Mar 31, 2022

Sider has detected 1 error on analyzing the commit 88faf88.

If the errors persist even after retrying, the following actions may resolve them:

If you still have problems, feel free to ask us via chat. 💬


You can turn off such notifications if unnecessary.

bgoonz added a commit that referenced this pull request Mar 31, 2022
Merge pull request #1879 from bgoonz/master
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.

3 participants