Skip to content

Pylint Exception logging #9077

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

Merged
l0lawrence merged 64 commits into from
Oct 10, 2024
Merged

Pylint Exception logging #9077

Changes from 1 commit
Commits
Show all changes
64 commits
Select commit Hold shift + click to select a range
9f84ad7
add placeholders
MJoshuaB Aug 11, 2024
c81ff56
add new placeholder comments
MJoshuaB Aug 20, 2024
18762be
exploring AST unfinished - minor emergency had to leave
Aug 21, 2024
2238161
identifying some mismatched functions
Aug 28, 2024
95fe015
refactor checker and tests
MJoshuaB Aug 28, 2024
e2e664f
fix error with non-builtins decorators
MJoshuaB Aug 28, 2024
46053e6
fine tuning and testing required
Aug 29, 2024
b402a5d
add pylint report
MJoshuaB Aug 29, 2024
3d051d7
add ranked listing of reports
MJoshuaB Aug 29, 2024
fdf6db7
format report as table
MJoshuaB Aug 29, 2024
fb160d5
add new verbs
MJoshuaB Sep 1, 2024
f06c915
update report
MJoshuaB Sep 2, 2024
db697b2
update reportcounts.md
MJoshuaB Sep 2, 2024
ae66d2c
fix formatting for reportcounts.md
MJoshuaB Sep 2, 2024
233f536
update reportcounts.md
MJoshuaB Sep 2, 2024
27edb80
minimal tests added
Sep 3, 2024
112173d
Merge pull request #2 from Azure/main
JessicaBell00 Sep 8, 2024
c5ffb91
Merge branch 'Azure:main' into main
JessicaBell00 Sep 10, 2024
7dc03ca
Merge branch 'working-main' into main
JessicaBell00 Sep 12, 2024
18f09e9
Merge pull request #5 from MJoshuaB/main
JessicaBell00 Sep 12, 2024
af098f7
Merge branch 'Azure:main' into working-main
JessicaBell00 Sep 12, 2024
cfba33d
Base code and unit tests
JessicaBell00 Sep 12, 2024
f7af031
Refactored class to be more specific
JessicaBell00 Sep 16, 2024
013a408
Added to README
JessicaBell00 Sep 16, 2024
fee5009
Merge branch 'main' of https://github.com/Azure/azure-sdk-tools into …
JessicaBell00 Sep 16, 2024
bec9467
Update README / fix merging
JessicaBell00 Sep 16, 2024
6e53b16
Final Refactor
Sep 17, 2024
687f542
Fixed false positives
JessicaBell00 Sep 18, 2024
80657b4
not running multiple times picking up on different function types
Sep 19, 2024
5b69e7e
Merge branch 'main' into invalid-use-of-@overload
16234397 Sep 19, 2024
b10a3cd
add TODOs
MJoshuaB Sep 21, 2024
a0e5c18
removed code not reached
Sep 24, 2024
bf6936c
Merge remote-tracking branch 'origin/invalid-use-of-@overload' into i…
Sep 24, 2024
3ed8120
Checks at a class level
Sep 24, 2024
ac9921a
Merge branch 'main' into invalid-use-of-@overload
16234397 Sep 24, 2024
4c638df
Looking into different connection_verify assignments
Sep 25, 2024
ab62179
Placeholders added back
16234397 Sep 26, 2024
795c861
Checker base done
Sep 26, 2024
35094ba
exclue private namespaces and classes
MJoshuaB Sep 26, 2024
ed5e211
update reports
MJoshuaB Sep 27, 2024
a835da4
Checker, Tests, & Readme done
Sep 29, 2024
693ea14
Update pylint_guidelines_checker.py
JessicaBell00 Sep 30, 2024
141da43
Fix another false positive
JessicaBell00 Sep 30, 2024
202faa2
Check either sides of indices
JessicaBell00 Oct 1, 2024
3abf67a
Merge branch 'main' of https://github.com/Azure/azure-sdk-tools into …
JessicaBell00 Oct 1, 2024
ae88cee
add new prefixes
MJoshuaB Oct 1, 2024
efe6ddc
Merge branch 'main' into working-main
MJoshuaB Oct 1, 2024
0bf7340
Merge branch 'working-main' into approved_prefix
MJoshuaB Oct 2, 2024
cfaa891
update unit tests
MJoshuaB Oct 2, 2024
a6f341a
remove reports
MJoshuaB Oct 2, 2024
5947f31
remove commented code
MJoshuaB Oct 2, 2024
65e431f
add checker to README
MJoshuaB Oct 2, 2024
c50a082
Merge branch 'working-main' into do-not-hardcode-boolean-connection_v…
16234397 Oct 5, 2024
24e9647
Merge pull request #10 from MJoshuaB/do-not-hardcode-boolean-connecti…
16234397 Oct 5, 2024
1f78550
Merge branch 'working-main' into invalid-use-of-@overload
16234397 Oct 5, 2024
56176ca
Merge pull request #11 from MJoshuaB/invalid-use-of-@overload
16234397 Oct 5, 2024
8cdce20
Tidy Up
Oct 5, 2024
6a319b7
Add TODO comment re other cases to investigate
JessicaBell00 Oct 6, 2024
7e7256b
Merge pull request #13 from Azure/main
JessicaBell00 Oct 6, 2024
6d262e3
Merge branch 'working-main' into approved_prefix
MJoshuaB Oct 6, 2024
18ea2fa
Merge pull request #12 from MJoshuaB/approved_prefix
MJoshuaB Oct 6, 2024
e2dcbb9
Merge branch 'working-main' into exception-logging
JessicaBell00 Oct 6, 2024
9027a8b
Revert "Merge branch 'working-main' into exception-logging"
JessicaBell00 Oct 6, 2024
9a32416
Make checker more explicit
JessicaBell00 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
Prev Previous commit
Next Next commit
update reportcounts.md
  • Loading branch information
@MJoshuaB
MJoshuaB committed Sep 2, 2024
commit db697b2e9d5e3338b76357029582de0b76916a3b
Loading