Dimitris Karagiannis

[This](https://github.com/Orfium/orfium-ictinus/blob/master/src/utils/initLocaleFormat.ts) should be inside a `useEffect` because otherwise it throws during server rendering

Speaking with http://ictinus.herokuapp.com/?path=/docs/design-system-button--icon-button-sizes as a reference, the `dataTestPrefixId` prop does not indicate that a default value is used (which in truth it's `button-`) Furthermore, even if `dataTestPrefixId` is passed a...

We should be able to just pass `data-testid`, which is the common convention (along with permutations `data-testId`, `data-testID` and `data-testiD` which are all valid attributes when using `...byTestId()` in `testing-library`)...

While it does update the `aria-checked` attribute it does not actually change the `checked` value. The `checked` value should be preferred over `aria-checked` since this is a native checkbox element....

Add JSDoc types and definitions

10

Added types to every function and config object, to the best of my ability. One thing that surfaced from this process is that in some places functions expect arguments as...

.out file in the repos directory, 0 results in results file

1

I tested this running locally, dockerized, on 2 repos from my org (let's call them A and B). Exact command was this: ``` docker run -v /home/mitch/Workspace/magpie-repos/:/repos -v /home/mitch/Documents/magpie/:/app/results punksecurity/secret-magpie...

fix: handle JS files when allowJs and checkJs options are set to true

1

This pr allows the tool to handle JS files as well when the `allowJs` and `checkJs` are explicitly enabled in the user's config. Resolves issues in the scenario when `tsc-files`...