ctubbsii commented on a change in pull request #2068:
URL: https://github.com/apache/accumulo/pull/2068#discussion_r624873723
##########
File path: README.md
##########
@@ -15,42 +15,97 @@ See the License for the specific language governing
permissions and
limitations under the License.
-->
-[![Apache Accumulo][logo]][accumulo]
---
-[![Build Status][ti]][tl] [![Maven Central][mi]][ml] [![Javadoc][ji]][jl]
[![Apache License][li]][ll]
-
-[Apache Accumulo][accumulo] is a sorted, distributed key/value store that
provides robust,
-scalable data storage and retrieval. With Apache Accumulo, users can store and
manage large
+<!-- LOGO -->
Review comment:
@nicolasalarconrapela We don't just use Markdown because it's shorthand
for HTML. We also use it because it's more human-readable. The less HTML, the
better. We don't need HTML. This README is also included in the tarball we
generate. So, it should be readable in a user's text editor, and the reader
should not have to parse HTML/XML tags to read it. Centering the image, in my
opinion, does not justify adding HTML that is harder for users to view using a
plain text viewer.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]