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 #3

Merged
merged 2 commits into from
Aug 19, 2022
Merged

Update README #3

merged 2 commits into from
Aug 19, 2022

Conversation

ssandino
Copy link
Member

This is a first draft to introduce a initial structure to the readme file.

This is a first draft to introduce a initial structure to the readme file.
README.md Outdated
@@ -1,2 +1,53 @@
# public
Public Repository of Social Income
<h1 align="center">
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: I think it's cleaner to not use html tags in markdown. I would replace it with proper markdown as you do in the sections afterwards

README.md Outdated
[:heart: Become a
sponsor](https://github.com/sponsors/san-socialincome)

<br />
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: remove html Tag

Cleaned up the readme file. The Social Income logo we can add later through markdown.
@ssandino
Copy link
Member Author

Good point. Pushed a new version (Missing logo: I will figure out later how to best display it with purely markdown)

@ssandino ssandino requested a review from andrashee August 19, 2022 07:54
@ssandino ssandino merged commit 63564b2 into main Aug 19, 2022
@ssandino ssandino deleted the readme-file-structure branch August 19, 2022 08:50
@ssandino ssandino restored the readme-file-structure branch August 19, 2022 11:11
@ssandino ssandino deleted the readme-file-structure branch September 2, 2022 14:37
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.

2 participants