Skip to content

Conversation

@scaredcat
Copy link
Contributor

Description of the Change

Change log entry for #810

@coveralls
Copy link

coveralls commented Mar 16, 2020

Pull Request Test Coverage Report for Build 1334

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 94.768%

Totals Coverage Status
Change from base Build 1332: 0.0%
Covered Lines: 1268
Relevant Lines: 1338

💛 - Coveralls

@auvipy auvipy merged commit fd0a0d8 into django-oauth:master Mar 17, 2020
@n2ygk
Copy link
Contributor

n2ygk commented Mar 17, 2020

@auvipy it seems like people are erasing the checklist from the pull_request_template. I ask that you please help enforce these items before approving and merging. Specifically, the author name is not in AUTHORS.

The checklist should have looked like this:

Checklist

  • PR only contains one change (considered splitting up PR)
  • unit-test added
  • documentation updated
  • CHANGELOG.md updated (only for user relevant changes)
  • author name in AUTHORS

@auvipy
Copy link
Contributor

auvipy commented Mar 18, 2020

noted! thanks!

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.

4 participants