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.

Sign up for GitHub

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

Jump to bottom

fix(docs): correct a couple links in the readme #22

Merged
merged 3 commits into from
Oct 3, 2023
Merged

fix(docs): correct a couple links in the readme #22

merged 3 commits into from
Oct 3, 2023

Conversation

rschristian
Copy link
Contributor

@rschristian rschristian commented Oct 3, 2023 •
edited
Loading

Component / Package Name: README.md

This PR contains:

  • bugfix
  • feature
  • refactor
  • documentation
  • other

Are tests included?

  • yes (bugfixes and features will not be merged without tests)
  • no

Breaking Changes?

  • yes (breaking changes will not be merged unless absolutely necessary)
  • no

List any relevant issue numbers:

N/A

Description

The link to the docs site (https://jsx.email/docs/) 404s and the contributing guide link had a different casing which also 404'd.

@rschristian rschristian changed the title docs: Correct link to docs site in readme docs: Correct a couple links in the readme Oct 3, 2023
@lordelogos lordelogos changed the title docs: Correct a couple links in the readme fix(docs): correct a couple links in the readme Oct 3, 2023
@lordelogos lordelogos merged commit 6efe016 into shellscape:main Oct 3, 2023
2 checks passed
@rschristian rschristian deleted the rschristian-patch-1 branch October 3, 2023 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rschristian @lordelogos