Skip to content

Conversation

jpierson-at-riis
Copy link
Contributor

Describe the Pull Request

Fix simple spelling mistake in message shown to user. The word "unpatched" was accidentally spelled as "upatched" which was most likely a typo in the original code.

Checklist for PR

  • Run MobSF unit tests and lint tox -e lint,test
  • Tested Working on Linux, Mac, Windows, and Docker
  • Add unit test for any new Web API (Refer: StaticAnalyzer/tests.py)
  • Make sure tests are passing on your PR MobSF tests

Additional Comments (if any)

DESCRIBE HERE

@ajinabraham
Copy link
Member

Thanks for the PR!

@ajinabraham ajinabraham requested a review from Copilot April 24, 2025 16:37
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request fixes a simple spelling mistake in the Android manifest description messages.

  • Fixed the improper spelling of "unpatched" which was mistakenly written as "upatched"
  • Updated messages in both 'title' and 'name' keys to reflect the correct spelling

@@ -64,7 +64,7 @@
},
'vulnerable_os_version': {
'title': ('App can be installed on a vulnerable '
'upatched Android version<br>Android %s, [minSdk=%s]'),
'unpatched Android version<br>Android %s, [minSdk=%s]'),
Copy link
Preview

Copilot AI Apr 24, 2025

Choose a reason for hiding this comment

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

Fixed the spelling mistake: changed 'upatched' to 'unpatched'.

Copilot uses AI. Check for mistakes.

@@ -73,7 +73,7 @@
'Support an Android version => 10, API 29 '
'to receive reasonable security updates.'),
'name': ('App can be installed on a vulnerable '
'upatched Android version %s, [minSdk=%s]'),
'unpatched Android version %s, [minSdk=%s]'),
Copy link
Preview

Copilot AI Apr 24, 2025

Choose a reason for hiding this comment

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

Fixed the spelling mistake: changed 'upatched' to 'unpatched'.

Copilot uses AI. Check for mistakes.

@ajinabraham ajinabraham merged commit 92ac55e into MobSF:master Apr 24, 2025
6 of 9 checks passed
J1-MI pushed a commit to J1-MI/OMT_Semi_project2_MobSF that referenced this pull request Sep 9, 2025
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.

2 participants