Skip to content

Remove version check locally - #2860

Merged
pombredanne merged 1 commit into
aboutcode-org:developfrom
adityasangave:local_version_check
Feb 14, 2022
Merged

pombredanne merged 1 commit into
aboutcode-org:developfrom
adityasangave:local_version_check

Conversation

@adityasangave

@adityasangave adityasangave commented Feb 9, 2022

Copy link
Copy Markdown
Contributor

Fixes #2796

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled 📑 and links the original issue above 🔗
  • Tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 📁

@adityasangave
adityasangave force-pushed the local_version_check branch 3 times, most recently from 9ae0389 to 0a0deae Compare February 9, 2022 13:30
Comment thread src/scancode/cli.py
if not outdated and check_version:
from scancode.outdated import check_scancode_version_remotely
outdated = check_scancode_version_remotely()
# Check for updates

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should ONLY happen if check_version. This way folks can elect to not do any such check

Signed-off-by: adii21-Ux <adityasangave2112@gmail.com>

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!
Thank you ++... merging.

@pombredanne
pombredanne merged commit a0e576a into aboutcode-org:develop Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bogus "Outdated ScanCode Toolkit version" message when running ScanCode 30.1.0

2 participants