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.

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

A few bug fixes and improvements for the next release #75

Merged
merged 7 commits into from Oct 27, 2017
Merged

Conversation

atodorov
Copy link
Member

These are all minor changes so I've bundled them together in one PR. See commit log.

test that user with the default permissions can view the comments
form widget when executing test cases.
this was caused by a bad import statement. Also added test for it.
the register account link is in the top-right menu, under
"Welcome Guest". This change adds a bit of text into the MOTD area
of the login screen to make registrations easier!
there's a new flake8 version and we have to ignore a few errors

E722: bare Except
E501: line too long
@coveralls
Copy link

coveralls commented Oct 27, 2017

Pull Request Test Coverage Report for Build 285

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.4%) to 58.081%

Totals Coverage Status
Change from base Build 279: 0.4%
Covered Lines: 7074
Relevant Lines: 11340

💛 - Coveralls

@atodorov atodorov merged commit 04ee633 into master Oct 27, 2017
@atodorov atodorov deleted the next_release branch October 27, 2017 10:13
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.

None yet

2 participants