Closed lorenzholzbauer closed 4 months ago
Path | Size | Loading time (3g) | Running time (snapdragon) | Total time |
---|---|---|---|---|
./kipper/web/kipper-standalone.min.js | 136.2 KB (0%) | 2.8 s (0%) | 3.2 s (+13.95% 🔺) | 5.9 s |
./kipper/web/kipper-standalone.js | 136.24 KB (0%) | 2.8 s (0%) | 2.9 s (+13.94% 🔺) | 5.6 s |
Attention: Patch coverage is 78.94737%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 83.45%. Comparing base (
ff5856d
) to head (8bedf39
).:exclamation: Current head 8bedf39 differs from pull request most recent head 444289f
Please upload reports for the commit 444289f to get more accurate results.
Files | Patch % | Lines |
---|---|---|
kipper/target-js/src/built-in-generator.ts | 0.00% | 4 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
What type of change does this PR perform?
Summary
Implemented string multiplication support for multiplicative expressions.
Closes #478
List of Changes
str * num
string multiplication.repeatString
which is used underneath for the string repetition behaviour.Does this PR create new warnings?
None.
Detailed Changelog
Added
*
operator. (#478).Linked issues or PRs