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

Update README.md #251

Merged
merged 1 commit into from
Dec 23, 2023
Merged

Update README.md #251

merged 1 commit into from
Dec 23, 2023

Conversation

phoropter
Copy link
Contributor

@phoropter phoropter commented Dec 11, 2023

Add text denoting mollysocket requirement for Molly-UP, unavailability for linked devices, and updated table.

@phoropter phoropter closed this Dec 11, 2023
@phoropter phoropter reopened this Dec 11, 2023
@BHydden
Copy link

BHydden commented Dec 11, 2023

In line 73 it might be good to directly state whether Molly-UP has blobs or not. The way it is written in your proposal I think leaves it ambiguous.

@phoropter
Copy link
Contributor Author

In line 73 it might be good to directly state whether Molly-UP has blobs or not. The way it is written in your proposal I think leaves it ambiguous.

Good point. I think the change makes it clear enough.

@BHydden
Copy link

BHydden commented Dec 12, 2023

Yeah sounds good thanks for that.

@phoropter
Copy link
Contributor Author

I think with 3 choices, a bullet list makes readability easier.

@BHydden
Copy link

BHydden commented Dec 12, 2023

I agree. Latest version is very clear ❤️

Add text denoting mollysocket requirement for Molly-UP, unavailability
for linked devices, and updated table.

Co-authored-by: Oscar Mira <[email protected]>
@valldrac
Copy link
Member

Thank you for the contribution. I've corrected some errors and made some changes to enhance clarity.

@valldrac valldrac merged commit 5e06706 into mollyim:main Dec 23, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants