f5-common-python icon indicating copy to clipboard operation
f5-common-python copied to clipboard

Fix Python SyntaxWarnings in iapp_parser.py

Open intgr opened this issue 2 years ago • 3 comments

If I run Python's compileall on code of the f5-sdk Python package, I get the following SyntaxWarnings:

% python3 -m compileall iapp_parser.py
Compiling 'iapp_parser.py'...
iapp_parser.py:123: SyntaxWarning: "is" with a literal. Did you mean "=="?
  if brace_count is 0:
iapp_parser.py:126: SyntaxWarning: "is not" with a literal. Did you mean "!="?
  if brace_count is not 0:

This PR fixes these warnings.

Using Python 3.11.2, but I believe these warnings were already introduced in Python 3.8.

Background: https://adamj.eu/tech/2020/01/21/why-does-python-3-8-syntaxwarning-for-is-literal/

intgr avatar Mar 14 '23 15:03 intgr

Solves part of #1554.

@zhaoqin-github Ping?

intgr avatar Mar 29 '23 12:03 intgr

Solves part of #1554.

@zhaoqin-github Ping?

Hi Marti,

As described in the README at main page, this python module is "NO LONGER UNDER ACTIVE DEVELOPMENT". I am not the maintainer of this project, and will not publish a new version for public usage. If you need this change, maybe you can manage it in your downstream repo and create package by yourself. Very sorry for that!

zhaoqin-github avatar Mar 30 '23 03:03 zhaoqin-github

Hi. Fair enough. I interpreted "not under active development" as in "we are no longer adding new features".

If you mean that this project is unmaintained -- bugs won't be fixed etc -- then I think the message could be clearer. Say it in the heading that this is unmaintained, and "issues and pull requests will not be handled".

intgr avatar Mar 30 '23 07:03 intgr