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]


Reply via email to