Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

README Revamp #4

Merged
merged 4 commits into from
Apr 7, 2024
Merged

Conversation

verysillycat
Copy link
Contributor

  • fully revamped README using almost the same format as i did with the Suncord README

  • fix releases URL

  • fully on Markdown not like last time

@verticalsync verticalsync requested review from verticalsync and removed request for verticalsync April 7, 2024 14:14
Copy link
Contributor Author

@verysillycat verysillycat left a comment

Choose a reason for hiding this comment

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

  • logo and badges & good looking discord badge
  • adds main features from Vesktop README but tweaked
  • and using repo's sunroof logo PNG not catbox.moe like last time

Copy link
Owner

@verticalsync verticalsync left a comment

Choose a reason for hiding this comment

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

.


You will get test errors about venmic, you can ignore these as it's a linux only module.
Copy link
Owner

Choose a reason for hiding this comment

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

Add this back into the installation part

README.md Show resolved Hide resolved

Find the required installer you need [here](https://github.com/verticalsync/Sunroof/releases/tag/latest).
Copy link
Owner

Choose a reason for hiding this comment

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

This never broke, and is the preferred way as latest in github goes to the most recently submitted

Copy link
Owner

Choose a reason for hiding this comment

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

nvm, i misread it. It has to be releases/latest not releases/tag/latest

building from source doesn't mean necessarily that you're going to contribute
@verticalsync verticalsync merged commit 9545896 into verticalsync:main Apr 7, 2024
1 check passed
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.

2 participants