[darknesspwner]

Description

I added f-strings instead of the legacy one using the modulo % in tools/

In simple terms. instead of writing:

print("checked: %d scripts, %d with problems, %d below size target" % (len(results), len(bad), len(short)))

You can just write:

print(f"checked: {len(results)} scripts, {len(bad)} with problems, {len(short)} below size target")

It does the exact same thing, but the second is more readable to everyone

Type of change

• New script
• Fix for an existing script
• Documentation
• Other

Script details (for new or changed scripts)

• Script name: lint_nse.py, smoke_nse.py
• Folder: /tools
• Risk level: None
• Tested against (lab or authorized target only, no real hostnames or IPs): None (the python scripts were tested)

Checklist

• [X ] Script runs without errors against a lab target I own or am authorized to test
• [ X] Script has a description, author, license, and categories field
• [ X] Risk level is accurately assessed
• [ X] python3 tools/lint_nse.py passes
• [ X] Folder README.md updated (and the root README.md Categories table for new folders)
• [ X] No hardcoded credentials, IPs, or destructive/intrusive actions
• [ X] No real hostnames, IPs or third-party data anywhere in the change
• [X ] Any write operation is documented and justified
• [ X] I wrote this contribution myself, with no AI-generated content
• [X ] No code copied from Nmap's bundled scripts or libraries, and I can release this under the MIT License

[NexzaDev]

Thanks for the contribution! Two fixes needed before I can merge:

1. tools/smoke_nse.py: the last print uses {faulries}, which is undefined. It should be {failures}. As written the script raises a NameError at the end of every run.
2. tools/lint_nse.py: the "MISSING/unknown Risk Level" message used %r. Please keep that behaviour with {risk!r}.

Please also run python3 tools/smoke_nse.py locally after the change and confirm it finishes without errors.

[darknesspwner]

Ok let me fix that real quick

Thanks for the contribution! Two fixes needed before I can merge:

1. tools/smoke_nse.py: the last print uses {faulries}, which is undefined. It should be {failures}. As written the script raises a NameError at the end of every run.

2. tools/lint_nse.py: the "MISSING/unknown Risk Level" message used %r. Please keep that behaviour with {risk!r}.

Please also run python3 tools/smoke_nse.py locally after the change and confirm it finishes without errors.