Skip to content
This repository was archived by the owner on Nov 25, 2024. It is now read-only.

chore: configured SvelteKit for Netlify#46

Merged
nickytonline merged 3 commits intonickytonline:mainfrom
gentlegiantdev:main
Oct 14, 2022
Merged

chore: configured SvelteKit for Netlify#46
nickytonline merged 3 commits intonickytonline:mainfrom
gentlegiantdev:main

Conversation

@gentlegiantdev
Copy link
Contributor

@gentlegiantdev gentlegiantdev commented Oct 13, 2022

What type of PR is this? (check all applicable)

  • Feature
  • Bug Fix
  • Documentation Update
  • Refactor
  • Optimization

Description

Removed "netlify.toml".

Related Tickets & Documents

QA Instructions, Screenshots, Recordings

Please replace this line with instructions on how to test your changes, a note
on the devices and browsers this has been tested on, as well as any relevant
images for UI changes.

UI accessibility concerns?

If your PR includes UI changes, please replace this line with details on how
accessibility is impacted and tested. For more info, check out the awesome
Forem Accessibility Docs.

Added/updated tests?

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

[optional] What gif best describes this PR or how it makes you feel?

alt_text

Copy link
Collaborator

@sadiejay sadiejay left a comment

Choose a reason for hiding this comment

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

I think this looks good to me! @nickytonline @Narigo

@nickytonline
Copy link
Owner

@sadiejay, it's all good based on my inital instructions in the issue, but I realized when testing this tonight, I clearly missed something. I dropped the updated instructions here, #45 (comment)

Copy link
Owner

@nickytonline nickytonline left a comment

Choose a reason for hiding this comment

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

Looks great @gentlegiantdev! Thanks so much for the pull request! 🚀

@nickytonline nickytonline merged commit 2d9f07d into nickytonline:main Oct 14, 2022
@nickytonline nickytonline changed the title chore: removed netlify.toml chore: configured SvelteKit for Netlify Oct 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Configure SvelteKit for Netlify

3 participants