Skip to content
This repository was archived by the owner on Dec 30, 2023. It is now read-only.

feat(runtime): ✨ Add a a new module module #34

Merged
merged 4 commits into from
Mar 16, 2023
Merged

Conversation

xeho91
Copy link
Member

@xeho91 xeho91 commented Mar 15, 2023

What is the purpose of this Pull Request?

Adds a new module module for snippets related to modules/dependencies/packages.

Type of this Pull Request

  • ✨ Implementing a new feature
  • 📝 Documentation update

@xeho91 xeho91 requested a review from a team March 15, 2023 16:30
@xeho91 xeho91 added the ✨ feature New feature label Mar 15, 2023
@xeho91 xeho91 self-assigned this Mar 15, 2023
@changeset-bot
Copy link

changeset-bot bot commented Mar 15, 2023

🦋 Changeset detected

Latest commit: a2b01b8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@terminal-nerds/snippets-runtime Minor
@terminal-nerds/snippets-function Patch
@terminal-nerds/snippets Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the 📓 documentation Improvements or additions to documentation label Mar 15, 2023
@ghost
Copy link

ghost commented Mar 15, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@github-actions
Copy link
Contributor

github-actions bot commented Mar 15, 2023

Size Change: +1.11 kB (+4%)

Total Size: 31.1 kB

Filename Size Change
./packages/runtime/dist/main.js 1.68 kB +370 B (+28%) 🚨
./packages/runtime/dist/module/module.js 740 B +740 B (new file) 🆕
ℹ️ View Unchanged
Filename Size
./packages/error/dist/built-in/built-in.js 350 B
./packages/error/dist/custom/custom.js 328 B
./packages/error/dist/main.js 514 B
./packages/extension/dist/main.js 481 B
./packages/extension/dist/schema/groups/html.js 201 B
./packages/extension/dist/schema/groups/javascript.js 221 B
./packages/extension/dist/schema/groups/json.js 207 B
./packages/extension/dist/schema/groups/markdown.js 210 B
./packages/extension/dist/schema/groups/stylesheet.js 221 B
./packages/extension/dist/schema/groups/typescript.js 221 B
./packages/extension/dist/schema/groups/yaml.js 202 B
./packages/extension/dist/schema/schema.js 483 B
./packages/function/dist/main.js 528 B
./packages/function/dist/schema/schema.js 530 B
./packages/regexp/dist/creator/creator.js 367 B
./packages/regexp/dist/main.js 631 B
./packages/runtime/dist/environment/environment.js 683 B
./packages/runtime/dist/scope/scope.js 1.01 kB
./packages/runtime/dist/variable/variable.js 937 B
./packages/snippets/dist/main.js 146 B
./packages/string/dist/case/case.js 87 B
./packages/string/dist/char/char.js 1.07 kB
./packages/string/dist/main.js 1.66 kB
./packages/string/dist/schema/schema.js 270 B
./packages/string/dist/word/word.js 270 B
./packages/test/dist/emoji/emoji.js 839 B
./packages/test/dist/main.js 2.59 kB
./packages/test/dist/sample/groups/array.js 126 B
./packages/test/dist/sample/groups/bigint.js 211 B
./packages/test/dist/sample/groups/boolean.js 188 B
./packages/test/dist/sample/groups/date.js 133 B
./packages/test/dist/sample/groups/function.js 709 B
./packages/test/dist/sample/groups/infinity.js 140 B
./packages/test/dist/sample/groups/map.js 152 B
./packages/test/dist/sample/groups/nan.js 119 B
./packages/test/dist/sample/groups/null.js 102 B
./packages/test/dist/sample/groups/number.js 262 B
./packages/test/dist/sample/groups/object.js 135 B
./packages/test/dist/sample/groups/regexp.js 126 B
./packages/test/dist/sample/groups/set.js 132 B
./packages/test/dist/sample/groups/string.js 327 B
./packages/test/dist/sample/groups/symbol.js 169 B
./packages/test/dist/sample/groups/undefined.js 119 B
./packages/test/dist/sample/sample.js 1.22 kB
./packages/test/dist/unit/unit.js 1.39 kB
./packages/type/dist/built-in/built-in.js 1.49 kB
./packages/type/dist/built-in/groups/control-abstraction.js 339 B
./packages/type/dist/built-in/groups/error.js 308 B
./packages/type/dist/built-in/groups/fundamental.js 262 B
./packages/type/dist/built-in/groups/indexed-collection.js 379 B
./packages/type/dist/built-in/groups/internationalization.js 224 B
./packages/type/dist/built-in/groups/keyed-collection.js 262 B
./packages/type/dist/built-in/groups/managing-memory.js 250 B
./packages/type/dist/built-in/groups/number-and-date.js 270 B
./packages/type/dist/built-in/groups/reflection.js 227 B
./packages/type/dist/built-in/groups/structured-data.js 291 B
./packages/type/dist/built-in/groups/test-processing.js 236 B
./packages/type/dist/constructor/constructor.js 334 B
./packages/type/dist/main.js 1.96 kB
./packages/type/dist/non-primitive/non-primitive.js 348 B
./packages/type/dist/primitive/primitive.js 504 B

compressed-size-action

@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 4433938838

  • 60 of 66 (90.91%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.2%) to 95.944%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/runtime/source/module/module.ts 60 66 90.91%
Totals Coverage Status
Change from base Build 4426594089: -0.2%
Covered Lines: 2086
Relevant Lines: 2138

💛 - Coveralls

@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit a2b01b8 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 90.9% (50% is the threshold).

This pull request will bring the total coverage in the repository to 97.5% (-0.2% change).

View more on Code Climate.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

88.3% 88.3% Coverage
0.0% 0.0% Duplication

@xeho91 xeho91 merged commit 40625dc into main Mar 16, 2023
@xeho91 xeho91 deleted the feat/runtime-module branch March 16, 2023 05:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
📓 documentation Improvements or additions to documentation ✨ feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants