GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — bpo-42238: [doc]: Hide false positive in make suspicious. by JulienPalard · Pull Request #29636 · python/cpython

Conversation

@JulienPalard

@JulienPalard JulienPalard commented Nov 19, 2021 •
edited by bedevere-bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

https://bugs.python.org/issue42238

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@JulienPalard
bpo-42238: [doc]: Hide false positive in make suspicious.
b01b0ea
@JulienPalard JulienPalard added docs Documentation in the Doc dir skip news labels Nov 19, 2021
@the-knights-who-say-ni the-knights-who-say-ni added the CLA signed label Nov 19, 2021
@bedevere-bot bedevere-bot added the awaiting core review label Nov 19, 2021
@JulienPalard JulienPalard added needs backport to 3.9 labels Nov 19, 2021
@JulienPalard

JulienPalard commented Nov 19, 2021

Copy link
Copy Markdown
Member Author

Original PR (#29389) has been backported to 3.9 and 3.10 so I'll backport this to 3.9 and 3.10 too.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@AlexWaygood

AlexWaygood commented Nov 19, 2021

Copy link
Copy Markdown
Member

Original PR (#29389) has been backported to 3.9 and 3.10 so I'll backport this to 3.9 and 3.10 too.

I think @ambv already applied this to 3.9/3.10:

  • dc23aed
  • 0111c5a

(Sorry for the bother — thanks for fixing!)

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@ambv ambv removed needs backport to 3.9 labels Nov 19, 2021
@ambv

ambv commented Nov 19, 2021

Copy link
Copy Markdown
Contributor

@JulienPalard, since we're no longer running make suspicious in CI for main per your own change (GH-26575), should we just stop caring about make suspicious at all?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@ambv
ambv merged commit d32316a into python:main Nov 19, 2021
@bedevere-bot bedevere-bot removed the awaiting core review label Nov 19, 2021
@JulienPalard
JulienPalard deleted the mdk-suspicious branch November 20, 2021 00:58
@JulienPalard

JulienPalard commented Nov 20, 2021 •
edited by bedevere-bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

@JulienPalard, since we're no longer running make suspicious in CI for main per your own change (GH-26575), should we just stop caring about make suspicious at all?

Soon, yes. The idea is I'm tracking closely make suspicious in bpo-42238 to see if there's true positives that I can implement in rstlint.py. It allowed me to add a few new things in rstlint:

  • bda69ab
  • fbf1008
  • 01331f1
  • b973542

and we do ensure rstlint in the CI as it make no false positives.

So « soon » rstlint should catch all the true positives of make suspicious, leaving only false positives for make suspicious.

This day I'll just drop it, and really stop caring about it.

Up to this day I prefer having an up-to-date susp-ignored.csv, so I can easily monitor new alerts.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@JulienPalard

JulienPalard commented Nov 20, 2021

Copy link
Copy Markdown
Member Author

(But I only watch master so backporting can be ignored, yes)

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@AlexWaygood

AlexWaygood commented Nov 20, 2021

Copy link
Copy Markdown
Member

and we do ensure rstlint in the CI as it make no false positives.

Arguably rstlint did have a few false positives in this PR 🙂 (but maybe I'm just too backtick-happy!)

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@JulienPalard

JulienPalard commented Nov 20, 2021

Copy link
Copy Markdown
Member Author

Arguably rstlint did have a few false positives in this PR slightly_smiling_face (but maybe I'm just too backtick-happy!)

This is a really interesting information, TIL, thanks! I'll have to look at it :D

AlexWaygood reacted with heart emoji
All reactions
  • ❤️ 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

remykarem pushed a commit to remykarem/cpython that referenced this pull request Dec 7, 2021
@JulienPalard remykarem
bpo-42238: [doc]: Hide false positive in make suspicious. (pythonGH-2…
70c231a
…9636)
@JulienPalard JulienPalard mentioned this pull request Apr 10, 2022
Deprecate suspicious.py? #86404
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@JulienPalard @AlexWaygood @ambv @the-knights-who-say-ni @bedevere-bot
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.