[Pki-devel] [PATCH] 95 -2 Fixes for comments on patch 95

So i addressed all the new comments from Fraser and Ade. Ade is OK with the current set of patches checked in (93, 94, 95-2). I will be submitting new patches which add the remaining methods in ProfileClient and provide a better solution for the property/setter usage in classes like ProfileData, CertEnrollmentRequest and CertReviewResponse. Fraser - if the current code is OK with you please let me know. i can check in the code right away. As per my previous mail - these are addressed: Though the link you provided is pretty old, i still do not see any support for showing the arguments of the wrapped function instead of the wrapper function's arguments - when calling help. In any case, i used functools.wraps to atleast show the docstring. Along with the following comments by Ade. 1. You need to not track workspace.xml - and most likely add it to .gitignore. See http://manski.net/2012/05/which-project-files-under-version-control/#inte... That said, it appears that Barbican for example, does not in fact check in their .idea files. You might want to ask them about that. I removed the PyCharm project files. (Even barbican doesn't check them in and they do not have any other way of checking the PEP8 standards using the PyCharm files) 2. In CertRevoke, you do check to see if the reason provided is a valid reason, but do this by defining the valid reasons twice - once as a standalone field, and once in an anonymous list. Better to define all the reasons in a named list, and check that list. -- Included in the patch. -- Abhishek