Skip to content

Commit

Permalink
Align logo, edit PR template (#6272)
Browse files Browse the repository at this point in the history
* Align badges and logo in README
* Comment out sections in PR template so they are only visible when creating a PR.
* fix typos in README
  • Loading branch information
fredster33 authored Sep 13, 2021
1 parent e64044b commit 9964d0d
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 9 deletions.
10 changes: 6 additions & 4 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
Thank you for submitting a contribution to Apache Geode.
<!-- Thank you for submitting a contribution to Apache Geode. -->

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
<!-- In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
-->

### For all changes:
- [ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
Expand All @@ -16,7 +17,8 @@ to ensure the following steps have been taken:

- [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)?

### Note:
<!-- Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to [email protected].
-->
2 changes: 1 addition & 1 deletion BUILDING.md
Original file line number Diff line number Diff line change
Expand Up @@ -103,5 +103,5 @@ The following steps have been tested with **IntelliJ IDEA 2020.3.3**
Some optional sanity tests to make sure things are working properly:
* Try looking up classes using **Navigate -> Class...**
* Open and run a distributed test such as BasicDistributedTest in geode-core.
* Create a new java class and ensure the Apache license is automatically added to the top of the
* Create a new Java class and ensure the Apache license is automatically added to the top of the
file with no blank line before the package line.
12 changes: 8 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,11 @@
[<img src="https://geode.apache.org/img/Apache_Geode_logo.png" align="center"/>](http://geode.apache.org)
<div align="center">

[![Apache Geode logo](https://geode.apache.org/img/Apache_Geode_logo.png)](http://geode.apache.org)

[![Build Status](https://concourse.apachegeode-ci.info/api/v1/teams/main/pipelines/apache-develop-main/badge)](https://concourse.apachegeode-ci.info/teams/main/pipelines/apache-develop-main) [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://www.apache.org/licenses/LICENSE-2.0) [![Maven Central](https://maven-badges.herokuapp.com/maven-central/org.apache.geode/geode-core/badge.svg)](http://search.maven.org/#search%7Cga%7C1%7Cg%3A%22org.apache.geode%22) [![homebrew](https://img.shields.io/homebrew/v/apache-geode.svg)](https://formulae.brew.sh/formula/apache-geode) [![Docker Pulls](https://img.shields.io/docker/pulls/apachegeode/geode.svg)](https://hub.docker.com/r/apachegeode/geode/) [![Total alerts](https://img.shields.io/lgtm/alerts/g/apache/geode.svg?logo=lgtm&logoWidth=18)](https://lgtm.com/projects/g/apache/geode/alerts/) [![Language grade: Java](https://img.shields.io/lgtm/grade/java/g/apache/geode.svg?logo=lgtm&logoWidth=18)](https://lgtm.com/projects/g/apache/geode/context:java) [![Language grade: JavaScript](https://img.shields.io/lgtm/grade/javascript/g/apache/geode.svg?logo=lgtm&logoWidth=18)](https://lgtm.com/projects/g/apache/geode/context:javascript) [![Language grade: Python](https://img.shields.io/lgtm/grade/python/g/apache/geode.svg?logo=lgtm&logoWidth=18)](https://lgtm.com/projects/g/apache/geode/context:python)

</div>

## Contents
1. [Overview](#overview)
2. [How to Get Apache Geode](#obtaining)
Expand All @@ -11,8 +15,8 @@
6. [Application Development](#development)
7. [Documentation](https://geode.apache.org/docs/)
8. [Wiki](https://cwiki.apache.org/confluence/display/GEODE/Index)
9. [How to Contribute?](https://cwiki.apache.org/confluence/display/GEODE/How+to+Contribute)
10. [Export Control](#export)
9. [How to Contribute](https://cwiki.apache.org/confluence/display/GEODE/How+to+Contribute)
10.[Export Control](#export)

## <a name="overview"></a>Overview

Expand Down Expand Up @@ -40,7 +44,7 @@ latency and 24x7 availability requirements.
You can download Apache Geode from the
[website](https://geode.apache.org/releases/), run a Docker
[image](https://hub.docker.com/r/apachegeode/geode/), or install with
[homebrew](https://formulae.brew.sh/formula/apache-geode) on OSX. Application developers
[Homebrew](https://formulae.brew.sh/formula/apache-geode) on OSX. Application developers
can load dependencies from [Maven
Central](https://search.maven.org/#search%7Cga%7C1%7Cg%3A%22org.apache.geode%22).

Expand Down

0 comments on commit 9964d0d

Please sign in to comment.