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 read me #264

Merged
merged 2 commits into from
Oct 26, 2024
Merged

Update read me #264

merged 2 commits into from
Oct 26, 2024

Conversation

lucadg911
Copy link
Contributor

Added section for mac users and an "disclosure" on the possible warning/error message due to ssl certificates not being loaded correctly.

@jaoleal
Copy link
Contributor

jaoleal commented Oct 26, 2024

Not a Mac user but seems good.

Maybe b1854f9 is not meant to be here. but on #265.

@lucadg911
Copy link
Contributor Author

Should I revert b1854f9 back to a read me without this clarification on this PR and perform this readme change on #265 ?

@Davidson-Souza
Copy link
Collaborator

I don't think we need to be this pedantic. It can fit here too, since it's a README update.

ACK b1854f9

@Davidson-Souza Davidson-Souza merged commit 62cd534 into vinteumorg:master Oct 26, 2024
6 checks passed
@lucadg911 lucadg911 deleted the update-read-me branch November 5, 2024 12:30
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