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

Remove unused dependency gatsby-plugin-remove-trailing-slashes #871

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

shrmnk
Copy link
Contributor

@shrmnk shrmnk commented Oct 11, 2024

Brief description of the changes

From the package docs: https://www.gatsbyjs.com/plugins/gatsby-plugin-remove-trailing-slashes/

Please Note: This plugin will soon be deprecated, please use Gatsby’s trailingSlash option. Read the documentation to learn more.

trailingSlash is already configured in gatsby-config.js (supported since [email protected]) and hence this plugin was not actually configured.

Type of changes

  • New device
  • Update existing device
  • Removing a device
  • General cleanup
  • Other

Checklist:

  • There are no passwords or secrets references in any examples.
    The only exceptions are !secret wifi_ssid and !secret wifi_password.
  • The wifi or ethernet block has no static / manual ip address specified.
  • The first configuration provided should be hardware definitions only.
    A more involved example can be provided in a separate configuration block.

Copy link

netlify bot commented Oct 11, 2024

Deploy Preview for esphome-devices ready!

Name Link
🔨 Latest commit c62dd4b
🔍 Latest deploy log https://app.netlify.com/sites/esphome-devices/deploys/6708743c7f48350008d581b7
😎 Deploy Preview https://deploy-preview-871--esphome-devices.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 93 (🔴 down 3 from production)
Accessibility: 86 (no change from production)
Best Practices: 100 (no change from production)
SEO: 82 (no change from production)
PWA: 70 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site 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.

1 participant