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

replace obsolete <p align=""> and added <img alt=""> in README.md #20553

Closed
wants to merge 3 commits into from

Conversation

abhishekmaity
Copy link

replace obsolete <p align=""> in accordance to HTML5 and added <img alt=""> in README.md

@abhishekmaity
Copy link
Author

@ef4 please review and help to merge the PR

ef4
ef4 previously approved these changes Oct 5, 2023
@abhishekmaity
Copy link
Author

@ef4 Can this PR be merged now.

@abhishekmaity
Copy link
Author

Please merge this PR

@kategengler kategengler dismissed ef4’s stale review October 14, 2023 14:08

Clearing review because of @dfreeman's good catch

@abhishekmaity
Copy link
Author

I'm closing it as the changes has different affect in the markdown.

@abhishekmaity abhishekmaity deleted the readme-fixes branch October 14, 2023 16:45
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