Skip to content

README updates#2913

Merged
dessalines merged 4 commits into
LemmyNet:mainfrom
mpusch88:main
Jun 5, 2023
Merged

README updates#2913
dessalines merged 4 commits into
LemmyNet:mainfrom
mpusch88:main

Conversation

@mpusch88
Copy link
Copy Markdown
Contributor

@mpusch88 mpusch88 commented Jun 4, 2023

Replaced unsecure link on lemmings with a Wikipedia article.

Commented out some missing links, added simple git contribution instructions for the time being.

Note - Contributing.md is also currently broken

mpusch88 added 2 commits June 4, 2023 13:16
…with wikipedia article.

Commented out sponsor list as page DNE.

Commented out Contributing section as all sites DNE. Added simple instructions for git contribution. Cleaned up formatting for Contributing section.

TODO: update join-lemmy.org/docs, uncomment removed lines
Comment thread README.md Outdated
- The old school [video game](<https://en.wikipedia.org/wiki/Lemmings_(video_game)>).
- The [Koopa from Super Mario](https://www.mariowiki.com/Lemmy_Koopa).
- The [furry rodents](http://sunchild.fpwc.org/lemming-the-little-giant-of-the-north/).
- The [furry rodents](https://en.wikipedia.org/wiki/Lemming).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is nothing wrong with this, a readonly page doesnt need https. And wikipedia is boring.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fair enough - considering this is a git readme it's probably not for general users.

as a note, i feel that having links with security errors on public facing sites can definitely scare away new users however!

Comment thread README.md Outdated
- [Support on Patreon](https://www.patreon.com/dessalines).
- [Support on OpenCollective](https://opencollective.com/lemmy).
- [List of Sponsors](https://join-lemmy.org/sponsors).
<!-- - [List of Sponsors](https://join-lemmy.org/sponsors). -->
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Dont comment it out but change to https://join-lemmy.org/donate

Comment thread README.md Outdated
- [Local Development](https://join-lemmy.org/docs/en/contributing/local_development.html)
- [Local Development](https://join-lemmy.org/docs/en/contributing/local_development.html) -->

- In order to contribute to Lemmy, you will need to fork this repository. You can then make changes to the code and submit a pull request. Changes will be reviewed and merged into the main repository if accepted.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fix the links instead of commenting them out. Text should go directly in the docs.

Copy link
Copy Markdown
Member

@dessalines dessalines left a comment

Choose a reason for hiding this comment

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

Thanks!

@dessalines dessalines merged commit ae88f61 into LemmyNet:main Jun 5, 2023
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