Piotr Idzik

style: do not suppress `this-escape`

1

This PR _includes_ `this-escape` into the active compiler warnings. The changes in the `*.java` files are _minimal_. Continuation of #5165. - [x] I have read [CONTRIBUTING.md](https://github.com/TheAlgorithms/Java/blob/master/CONTRIBUTING.md). - [x] This pull...

[](https://gitpod.io/from-referrer/) [know more](https://www.gitpod.io/docs/pull-requests/) ### Describe your change: This PR adds missing tests and handles _wrong_ inputs in `HexToDecimal`. ### Checklist: - [x] I have read [CONTRIBUTING.md](https://github.com/TheAlgorithms/Javascript/blob/master/CONTRIBUTING.md). - [x] This...

[](https://gitpod.io/from-referrer/) [know more](https://www.gitpod.io/docs/pull-requests/) ### Describe your change: I added tests for `NumberOfSubsetEqualToGivenSum`. The current implementation does not handle properly inputs less or equal `0`, so I decided to explicitly...

[](https://gitpod.io/from-referrer/) [know more](https://www.gitpod.io/docs/pull-requests/) ### Describe your change: - [x] Adds missing tests. ### Checklist: - [x] I have read [CONTRIBUTING.md](https://github.com/TheAlgorithms/Javascript/blob/master/CONTRIBUTING.md). - [x] This pull request is all my own...

Since `override` and `final` are [keywords](https://en.cppreference.com/w/cpp/keyword), they deserve to be _highlighted_ as code, similarly as `double` and `float` in [here](https://github.com/cpp-best-practices/cppbestpractices/blob/master/08-Considering_Performance.md#prefer-double-to-float-but-test-first) or `assert` in [here](https://github.com/cpp-best-practices/cppbestpractices/blob/master/05-Considering_Maintainability.md#never-use-assert-with-side-effects). Yes, this PR is just a...

Same as TheAlgorithms/Rust#647 - it contains the description of the problem, which this PR solves.

[BUG] CI is failing due to wrong spelling

1

### Bug Description I noticed that the CI-push is reporting this spelling errors: - https://github.com/juspay/hyperswitch/actions/runs/11135615914/job/30945965216, - https://github.com/juspay/hyperswitch/actions/runs/11123807888/job/30908038374, - https://github.com/juspay/hyperswitch/actions/runs/11123440265/job/30906897289, - https://github.com/juspay/hyperswitch/actions/runs/11123158701/job/30906005608. #6194 attempts to fix the spelling. ### Expected Behavior...

chore(router): use proper spelling in cloneable

1

## Type of Change - [x] Bugfix - [ ] New feature - [ ] Enhancement - [ ] Refactoring - [ ] Dependency updates - [ ] Documentation -...

feat(router): add limit to `generic_user_roles_list_for_org_and_extra`

1

## Type of Change - [ ] Bugfix - [x] New feature - [x] Enhancement - [ ] Refactoring - [ ] Dependency updates - [ ] Documentation - [...

style: declare single argument constructors as `explicit`

2

This PR declares all of the single argument constructors as `explicit`. This makes the _implicit conversion_ impossible. Cf. - [C++ best practices](https://github.com/cpp-best-practices/cppbestpractices/blob/master/03-Style.md#single-parameter-constructors), - [iso c++](https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c46-by-default-declare-single-argument-constructors-explicit).