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

Update linters and adjust code #132

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
f30d939
remove old linters and scripts
sergioteula Oct 5, 2024
5089686
added pre-commit
sergioteula Oct 5, 2024
19c9122
ruff and mypy working for precommit
sergioteula Oct 6, 2024
cb65227
added ruff mypy and tests to github workflow
sergioteula Oct 6, 2024
70725b9
adjust pyproject and remove workflow for pull request
sergioteula Oct 6, 2024
d76c0ef
try fix for tests with other python versions
sergioteula Oct 6, 2024
7559ebb
use target version for ruff
sergioteula Oct 6, 2024
dfcd6c8
fix unittest execution
sergioteula Oct 6, 2024
e04ed75
test python 3.5 for tests
sergioteula Oct 6, 2024
d72d924
removed python 3.5 for tests
sergioteula Oct 6, 2024
51c3294
added makefile
sergioteula Oct 6, 2024
6d5cdda
automatic fixes from pre-commit
sergioteula Oct 7, 2024
bb07147
applied ruff automatic fixes
sergioteula Oct 7, 2024
8b5dcea
added command to run all tests with different python versions
sergioteula Oct 7, 2024
0f27d27
added pre-commit for mypy from repo
sergioteula Oct 8, 2024
63f6002
use pytest for running tests
sergioteula Oct 8, 2024
ecd790f
add coverage to py312 test job
sergioteula Oct 8, 2024
a97c17e
add pytest config
sergioteula Oct 8, 2024
f1468d4
added environment file for credentials in tests
sergioteula Oct 8, 2024
33649e3
add short summary for pytest
sergioteula Oct 9, 2024
244c189
use amazon credentials on workflow
sergioteula Oct 9, 2024
37ad1fa
created integration test for varios api methods
sergioteula Oct 9, 2024
ae44f9d
run tests in series
sergioteula Oct 9, 2024
afc2a5b
remove get variations from test
sergioteula Oct 9, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
remove old linters and scripts
sergioteula committed Oct 5, 2024
commit f30d939d8a7026164ed7661add704f73b3b694c7
4 changes: 0 additions & 4 deletions .flake8

This file was deleted.

Empty file added .githooks/pre-commit
Empty file.
12 changes: 0 additions & 12 deletions .githooks/pre-push

This file was deleted.

3 changes: 0 additions & 3 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,11 +1,8 @@
# Custom folders and files to ignore
.vscode/
.DS_Store
secrets.py
test.py
coverage_html_report


# Byte-compiled / optimized / DLL files
__pycache__/
*.py[cod]
3 changes: 0 additions & 3 deletions .shellcheckrc

This file was deleted.

2 changes: 1 addition & 1 deletion LICENSE
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
MIT License

Copyright (c) 2020 Sergio Abad
Copyright (c) 2024 Sergio Abad

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
24 changes: 0 additions & 24 deletions pyproject.toml
Original file line number Diff line number Diff line change
@@ -1,24 +0,0 @@
[tool.black]
preview = true
exclude = ".*/sdk/.*"

[tool.isort]
profile = "black"
skip_glob = "*/sdk/*"

[tool.pylint]
[tool.pylint.master]
ignore = ["test.py"]
ignore-paths = [".*/sdk/", ".*docs/"]
[tool.pylint.message_control]
disable = [
"no-self-use",
"protected-access",
"too-many-arguments",
"too-many-instance-attributes",
"too-many-locals",
"too-many-public-methods",
]
ignored-argument-names = "args|kwargs"
[tool.pylint.similarities]
ignore-imports = true
8 changes: 8 additions & 0 deletions requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
certifi
mypy
pre-commit
python_dateutil
ruff
setuptools
six
urllib3
23 changes: 0 additions & 23 deletions scripts/check_black

This file was deleted.

23 changes: 0 additions & 23 deletions scripts/check_flake8

This file was deleted.

23 changes: 0 additions & 23 deletions scripts/check_isort

This file was deleted.

23 changes: 0 additions & 23 deletions scripts/check_pylint

This file was deleted.

30 changes: 0 additions & 30 deletions scripts/helpers

This file was deleted.

24 changes: 0 additions & 24 deletions scripts/run_tests

This file was deleted.