feat(TypeScript): ltrim

[DimensionalDragon]

• Programming langugage: TypeScript

• Function(s) that I'm implementing: ltrim

• Issue number: Typescript Implementation #18

• I have created unit tests to support my implementation

• I have created in-code documentation (or docstring) to document my code

• (Optional) I have updated the README.md file to document the function I'm implementing

[codecov]

Codecov Report

Base: 100.00% // Head: 100.00% // No change to project coverage 👍

Coverage data is based on head (29835d5) compared to base (a8f30c7).
Patch coverage: 100.00% of modified lines in pull request are covered.

❗ Current head 29835d5 differs from pull request most recent head 900a362. Consider uploading reports for the commit 900a362 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##            master       #71    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           55         9    -46     
  Lines          732        72   -660     
  Branches        20        23     +3     
==========================================
- Hits           732        72   -660     

Flag	Coverage Δ
clojure	?
csharp	?
go	?
rust	?
typescript	100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
typescript/src/string/count_chars.ts	100.00% <100.00%> (ø)
typescript/src/string/ltrim.ts	100.00% <100.00%> (ø)
csharp/Pehape/String/Explode.cs
spec/pehape/string/str_repeat_spec.rb
rust/src/string/implode.rs
go/strchr.go
clojure/src/pehape/string/implode.clj
rust/tests/string_test/explode_test.rs
csharp/Pehape/String/Md5.cs
spec/pehape/string/str_rot13_spec.rb
... and 39 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[krowter]

Btw, in character class you only need to escape backslashes and closing brackets
const charactersEscaped = characters.replace('\\', '\\\\').replace(']', '\\]')
easier to read too :)

[DimensionalDragon]

Oh, right. I forgot to change that, I originally used the (a|b|c) implementation where every special characters must be escaped, but I forgot to change it when I change the implementation to [abc]. Thanks for reminding me.

[lgtm-com]

This pull request introduces 2 alerts when merging 900a362 into a8f30c7 - view on LGTM.com

new alerts:

• 2 for Incomplete string escaping or encoding