Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[misc] Various cleanup #456

Merged
merged 1 commit into from
Aug 18, 2020
Merged

[misc] Various cleanup #456

merged 1 commit into from
Aug 18, 2020

Conversation

Djailla
Copy link

@Djailla Djailla commented Aug 15, 2020 •
edited
Loading

  • Adding in AUTHORS file
  • Fix minimal Django version in setup
  • Remove deprecated options in travis config

@codecov
Copy link

codecov bot commented Aug 15, 2020 •
edited
Loading

Codecov Report

Merging #456 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #456   +/-   ##
=======================================
  Coverage   88.81%   88.81%           
=======================================
  Files          34       34           
  Lines        2601     2601           
=======================================
  Hits         2310     2310           
  Misses        291      291

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8f12362...c512759. Read the comment docs.

@yakky
Copy link
Member

yakky commented Aug 18, 2020 •
edited
Loading

This has been integrated as part of #452

@Djailla
Copy link
Author

Djailla commented Aug 18, 2020

@yakky nope. This is a new PR

yakky
yakky requested changes Aug 18, 2020
View reviewed changes
Copy link
Member

@yakky yakky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I only skimmed through it before

tox.ini Outdated Show resolved Hide resolved
AUTHORS.rst Outdated Show resolved Hide resolved
[misc] Various cleanup
c512759 
- Adding in AUTHORS file
- Fix minimal Django version in setup
- Remove deprecated options in travis config
@chrisglass chrisglass merged commit fd300df into jazzband:master Aug 18, 2020
@chrisglass
Copy link
Collaborator

Thanks!

@Djailla Djailla deleted the misc branch August 18, 2020 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yakky yakky yakky approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Djailla @yakky @chrisglass