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

[TS-171] pylint: Module 'http.server' has no 'ThreadingHTTPServer' member #98

Open
mirabilos opened this issue Jan 12, 2023 · 7 comments

Comments

@mirabilos
Copy link

Erm, it does. That’s what I run.

@github-actions github-actions bot changed the title pylint: Module 'http.server' has no 'ThreadingHTTPServer' member [TS-171] pylint: Module 'http.server' has no 'ThreadingHTTPServer' member Jan 12, 2023
@github-actions
Copy link

Internal ticket created : TS-171

@DMarinhoCodacy
Copy link
Contributor

Hello @mirabilos

Thanks for reporting this.

Are you still experiencing the same behavior?

Kind regards,
David Marinho

mirabilos added a commit to qvest-digital/jensjs that referenced this issue May 30, 2023
mirabilos added a commit to qvest-digital/jensjs that referenced this issue May 30, 2023
@mirabilos
Copy link
Author

Yes, it shows still up https://github.com/tarent/jensjs/security/code-scanning/671, together with even more error messages that are not even errors

@DMarinhoCodacy
Copy link
Contributor

Hello @mirabilos,

I've made some tests with http.server and ThreadingHTTPServer is not returning any error since we bumped Pylint to a recent version (not the latest one yet but we'll do that very soon). But, either way, you're completely right. According to the docs, ThreadingHTTPServer is part of http.server since Python3.7.

Can you take some of your time and test if you still have this error?

Thanks in advance for your help.

Kind regards,
David Marinho

mirabilos added a commit to qvest-digital/jensjs that referenced this issue Oct 3, 2023
@mirabilos
Copy link
Author

Ah, unfortunately it seems I cannot retest this, as it fails to run nowadays:

Error: Code Scanning could not process the submitted SARIF file:
rejecting SARIF, as there are more runs than allowed (21 > 20)

I think this is because too many test utilities run, or something.

@mirabilos
Copy link
Author

Ah hmm. https://docs.codacy.com/repositories-configure/codacy-configuration-file/#legacy-pylint-19 says to disable pylint and rely on pylintpython3.

Which is what I want, actually.

mirabilos added a commit to qvest-digital/jensjs that referenced this issue Jan 20, 2024
@mirabilos
Copy link
Author

However, this…

$ cat .codacy.yml                                                               
---
engines:
  pylint:
    # pylintpython3 is used instead
    enabled: false
  tsqllint:
    # for Microsoft only
    enabled: false

… does not disable either ☹☹☹

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants