Sami Daniel comments

Results 16 comments of


                                            Sami Daniel

`sdiff` command is not implemented

taking this up

matches/exec: create path integrity check before execution

> could you also please add tests (with error mgmt)? thanks done

matches/exec: create path integrity check before execution

> > could you also please add tests (with error mgmt)? thanks > > done Ok, I will fix these errors that are occurring on Windows before sending another push.

matches/exec: create path integrity check before execution

Since we cannot test in the exec.rs file, I couldn't test (directly) the `check_path_integrity` function. So, the codecov will probably point out an error at this point. However, tests have...

Create the side-by-side option (-y) feature for the diff command (Incomplete)

I'll fix the alerts before reopening the PR, 😁

Create the side-by-side option (-y) feature for the diff command (Incomplete)

Is done : )

Create the side-by-side option (-y) feature for the diff command (Incomplete)

@sylvestre I think there is now a cool version of the side. Can you take a look? : )

Fix Prefix Matching Logic in PrefixContainer

> @sami-daniel please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information. > > ``` > @dotnet-policy-service agree [company="{your company}"] >...

Fix Prefix Matching Logic in PrefixContainer

> A small JIT-trick for better machine code. I didn't know that, but it makes sense, thanks!

Fix Prefix Matching Logic in PrefixContainer

> > I didn't know that > > The nice thing on contributing: you help to improve the product and get the opportunity to learn something new. A win -...