[PATCH] 69 Last set of fixes for warnings and errors reported by pylint for Dogtag10.1 branch
- First Post
- Replies
- Stats
-
Go to
- ----- 2024 -----
- May
- April
- March
- February
- January
- ----- 2023 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2022 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2021 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2020 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2019 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2018 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2017 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2016 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2015 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2014 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2013 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2012 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2011 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2010 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2009 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2008 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
Please review the patch with fixes for remaining two issues reported by
pylint.
1. Attribute defined outside init.
2. Instance attribute hiding a class method (Ignored as the problem is
in json.encoder.JSONEncoder).
Also attached the final report.
--Abhishek
Attachments:
- pylint-report-final.ksh (text/plain — 4.9 KB)
- pki-akoneru-0069-Fixes-for-issues-reported-by-pylint.patch (text/x-patch — 4.8 KB)
ACK
Extraneous E202 in comment in encoder class.
As discussed, please add more description of the E202 problem in
encoder.pl so we know why we are ignoring it.
Ade
On Wed, 2013-07-17 at 16:38 -0400, Abhishek Koneru wrote:
Please review the patch with fixes for remaining two issues reported
by
pylint.
1. Attribute defined outside init.
2. Instance attribute hiding a class method (Ignored as the problem is
in json.encoder.JSONEncoder).
Also attached the final report.
--Abhishek
_______________________________________________
Pki-devel mailing list
Pki-devel(a)redhat.com
https://www.redhat.com/mailman/listinfo/pki-devel
Added the description and removed the extraneous E0202 in the comment.
Pushed to master.
-Abhishek
On Thu, 2013-07-18 at 10:47 -0400, Ade Lee wrote:
ACK
Extraneous E202 in comment in encoder class.
As discussed, please add more description of the E202 problem in
encoder.pl so we know why we are ignoring it.
Ade
On Wed, 2013-07-17 at 16:38 -0400, Abhishek Koneru wrote:
> Please review the patch with fixes for remaining two issues reported by
> pylint.
>
> 1. Attribute defined outside init.
> 2. Instance attribute hiding a class method (Ignored as the problem is
> in json.encoder.JSONEncoder).
>
> Also attached the final report.
>
> --Abhishek
> _______________________________________________
> Pki-devel mailing list
> Pki-devel(a)redhat.com
> https://www.redhat.com/mailman/listinfo/pki-devel
3940
days inactive
3941
days old
2 comments
2 participants
tags (0)
participants (2)
-
Abhishek Koneru -
Ade Lee