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

Feature/linter fixes #27

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

martin-marty
Copy link
Contributor

Fixes for mypy and pylint

Github workflow added to run on push

Squashed commit of the following:

commit 6446d4d
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 14:12:13 2022 +0200

	Remove re-use database

commit a5f2dd4
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 14:00:15 2022 +0200

	Add additional root password for container

commit 7c8cc22
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:56:05 2022 +0200

	MySQL password

commit 37b9282
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:53:23 2022 +0200

	MySQL environment names

commit 8fd5223
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:51:01 2022 +0200

	Correct environment variable names

commit 5fd2a25
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:44:57 2022 +0200

	MySQL credentials

commit bdb4635
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:38:50 2022 +0200

	PG config

commit da2dff3
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:36:22 2022 +0200

	Postgres configuration

commit 4023b2f
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:34:34 2022 +0200

	Services test

commit 88574f7
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:29:21 2022 +0200

	Testing postgres

commit 3a44730
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:23:06 2022 +0200

	Credentials to environment

commit a85c862
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:15:37 2022 +0200

	Mysql port

commit 081e3c2
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:15:28 2022 +0200

	Debug mysql not starting

commit c294fa6
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:12:02 2022 +0200

	Correct init name

commit 224ffa9
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:10:53 2022 +0200

	Space

commit e45b806
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:10:17 2022 +0200

	Add postgres

commit 0d06d4f
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:03:53 2022 +0200

	Start MySQL

commit 23a3428
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 13:00:01 2022 +0200

	Missed -

commit 057155a
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:59:12 2022 +0200

	Bad line

commit bea8f1c
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:58:34 2022 +0200

	Correct spacing

commit dde3da5
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:55:52 2022 +0200

	Add mysql to action

commit d4dfd22
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:51:22 2022 +0200

	Tox tests

commit 55e651e
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:43:12 2022 +0200

	Gh tox added

commit b7e77d7
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:35:47 2022 +0200

	Strip tests back

commit 20e3cfe
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:34:50 2022 +0200

	Correct workflow used

commit 4d264a0
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:33:29 2022 +0200

	Initial test runner using tox

commit 965a331
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:33:15 2022 +0200

	Remove ignore all

commit ade64e2
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:20:55 2022 +0200

	Update tox, django >3.2, py>3.9

commit ef62934
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:20:25 2022 +0200

	Update classifiers and install_requires

commit 96e8c5d
Author: Martin Whitehouse <[email protected]>
Date:   Thu Aug 11 12:19:45 2022 +0200

	Drop django <3.2 from requirements
- mypy finding no issues
- pylint score raised from ~3 to ~6 mostly docs missing
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.

1 participant