Added release badge to the Readme and updated contributions! #2081

Merged
ssiddhantsharma merged 2 commits from ssiddhantsharma-patch-1 into master 2018-11-01 22:33:31 +01:00
ssiddhantsharma commented 2018-10-31 18:38:29 +01:00 (Migrated from github.com)

PR Checklist

Please check all that apply to this PR using "x":

  • I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
  • I have checked that this PR does not introduce a breaking change
  • This PR introduces breaking changes and I have provided a detailed explanation below

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Breaking changes (bugfix or feature that introduces breaking changes)
  • Code style update (formatting)
  • Refactoring (no functional changes)
  • Documentation changes
  • Other - Please describe:

Fixes

Issue Number: N/A

What is the current behavior?

What is the new behavior?

Other information

## PR Checklist Please check all that apply to this PR using "x": - [x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged) - [x] I have checked that this PR does not introduce a breaking change - [x] This PR introduces breaking changes and I have provided a detailed explanation below ## PR Type What kind of change does this PR introduce? <!-- Please check all that apply to this PR using "x". --> - [ ] Bugfix - [ ] Feature - [ ] Breaking changes (bugfix or feature that introduces breaking changes) - [ ] Code style update (formatting) - [ ] Refactoring (no functional changes) - [x] Documentation changes - [ ] Other - Please describe: ## Fixes Issue Number: N/A ## What is the current behavior? ## What is the new behavior? ## Other information <!-- If this PR contains a breaking change, please describe the impact and solution strategy for existing applications below. -->
neb-b commented 2018-10-31 18:44:35 +01:00 (Migrated from github.com)

Thanks for the PR @ssiddhantsharma

Could you move the release badge to #2 in the list? Right next to the license badge. I also think we can remove the code quality badge.

Thanks for the PR @ssiddhantsharma Could you move the `release` badge to #2 in the list? Right next to the license badge. I also think we can remove the `code quality` badge.
ssiddhantsharma commented 2018-11-01 19:17:47 +01:00 (Migrated from github.com)

@seanyesmunt I have updated with required changes!

@seanyesmunt I have updated with required changes!
neb-b commented 2018-11-01 22:33:27 +01:00 (Migrated from github.com)

Awesome! Thanks 🙂

Awesome! Thanks 🙂
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#2081
No description provided.