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

Overhaul README #522

Merged
merged 5 commits into from
Feb 8, 2021
Merged

Overhaul README #522

merged 5 commits into from
Feb 8, 2021

Conversation

tneotia
Copy link
Collaborator

@tneotia tneotia commented Feb 1, 2021

I decided to give the README a refresh since the last major update to it was in 2019 and a lot has been changed and updated since then. I hope I got all the details correct, if not a reviewer can probably fix it.

@tneotia
Copy link
Collaborator Author

tneotia commented Feb 1, 2021

@ryan-berger I noticed on #286 you were hesitant to include CSS attributes in the README, if this is still the case I can remove that section.

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@tneotia tneotia requested a review from erickok February 2, 2021 11:51
@tneotia
Copy link
Collaborator Author

tneotia commented Feb 4, 2021

@ryan-berger updated README to add customImageRender details in preparation for @erickok 's PR. If anything seems off let me know and I'll correct it.

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