Sanket Saurav

Fix some bug risks and code quality issues

7

Changes: - In `hammer/library/aws/s3.py`, use `in` operator to check if variable is equal to either of two values. - In `hammer/tools/ddb_inject_credentials.py`, use `is` to compare with `None`. - In `hammer/reporting-remediation/bot/commands.py`,...

Hi! 👋 I'm the founder of [DeepSource](https://deepsource.io), a code review automation tool that detects problems in every PR using static analysis and helps developers save a ton of time during...

This PR adds`.deepsource.toml` to run static analysis continuously on the repo with [DeepSource](https://deepsource.io). Upon enabling DeepSource, static analysis will be run on every PR to detect 650+ types of issues...

This PR adds a customized `.deepsource.toml` file for this project. Upon enabling DeepSource, static analysis will be run on every PR to detect 550+ types of issues in the changes...

Add support for rectangle area crop

40

Would love it if you could include support for rectangular area crop, in addition to the existing circle and square.

In addition to the default set of Octicons, there should be a way to register new SVG icons. Using the new icons should be identical to how default icons are...

Changes: This PR fixes multiple occurrences of the following code quality issues found by DeepSource's Autofix™ AI on this project. Issues: - Unnecessary use of dict() - Handle unguarded `next()`...

It is recommended to follow the Python naming convention while naming things. Reference: https://deepsource.io/gh/sanketsaurav/s3tree/issue/PYL-C0103/description

## Description `raise` causes control flow to be disrupted, as it will exit the block. You may check other conditions using another if statement. ## Occurrences https://deepsource.io/gh/sanketsaurav/s3tree/issue/PYL-R1720/description