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

README: Add snap as an installation option #544

Merged
merged 3 commits into from Jan 23, 2020

Conversation

ppd
Copy link
Member

@ppd ppd commented Jan 23, 2020

No description provided.

README.md Show resolved Hide resolved
README.md Outdated
@@ -39,6 +39,20 @@ and cannot guarantee their functionality.

[notesalexp]: https://notesalexp.org/packages/en/source/solvespace/

### Via Snap Store

Builds from master are automatically released to the `edge` channel in the Snap Store. Those packages receive no manual scrutiny and do not constitute official releases.
Copy link
Contributor

Choose a reason for hiding this comment

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

In fact, the master branch of SolveSpace is generally quite stable and well suited for daily use in spite of occasional bugs. We don't have to warn so strongly against it. I would say something like...

Those packages contain the latest improvements, but receive less testing than release builds.

Copy link
Contributor

Choose a reason for hiding this comment

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

There's not been a release for very long for various regrettable reasons, and 2.3 is very outdated, so de facto, master is currently what I recommend using to many people. But that's not a very satisfying situation either, so let's not just say that outright in README.

Copy link
Member Author

Choose a reason for hiding this comment

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

Less harsh words + the future prospects of an official release seems like a nice compromise in this section

@whitequark whitequark merged commit 72bc71c into solvespace:master Jan 23, 2020
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.

None yet

2 participants