Review of "Kill Port" version 2

Details Page Preview

Extension to easly kill all process on port


No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

All Versions

Version Status
4 Active
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
Thanks for removing .git folder. Since the end user don't know the entry value is going directly to the terminal command, you need to make sure `entry.get_text()` is always a valid integer before spawning the command line. To do that I recommend to use parseInt(): https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/parseInt Also check whether the parseInt() result isn't NaN. Please join us on GNOME Matrix channel if you have any questions related to the extensions: https://matrix.to/#/#extensions:gnome.org