Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: adding some type check #217

Merged
merged 13 commits into from
Apr 12, 2024
Merged

feat: adding some type check #217

merged 13 commits into from
Apr 12, 2024

Conversation

JeffreyDallas
Copy link
Contributor

@JeffreyDallas JeffreyDallas commented Apr 9, 2024

Description

This pull request changes the following:

  • adding some type check

Related Issues

@JeffreyDallas JeffreyDallas requested review from leninmehedy and a team as code owners April 9, 2024 17:36
@JeffreyDallas JeffreyDallas marked this pull request as draft April 9, 2024 17:37
github-actions bot and others added 3 commits April 9, 2024 17:42
…type-check

Signed-off-by: Jeffrey Tang <[email protected]>

# Conflicts:
#	README.md
Signed-off-by: Jeffrey Tang <[email protected]>
Copy link
Contributor

github-actions bot commented Apr 9, 2024

Unit Test Results

  1 files  ±0   17 suites  ±0   1m 37s ⏱️ +15s
114 tests ±0  114 ✅ ±0  0 💤 ±0  0 ❌ ±0 
139 runs  ±0  139 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit d77cfb1. ± Comparison against base commit d1017d8.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Apr 9, 2024

E2E Test Results

 1 files  ±0  10 suites  ±0   14m 42s ⏱️ +30s
84 tests ±0  84 ✅ ±0  0 💤 ±0  0 ❌ ±0 
86 runs  ±0  86 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit a81d56c. ± Comparison against base commit 07f5fe4.

♻️ This comment has been updated with latest results.

Copy link

codacy-production bot commented Apr 10, 2024

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-0.31% (target: -1.00%) 38.46%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (07f5fe4) 2989 2334 78.09%
Head commit (a81d56c) 3019 (+30) 2348 (+14) 77.77% (-0.31%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#217) 26 10 38.46%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

You may notice some variations in coverage metrics with the latest Coverage engine update. For more details, visit the documentation

Copy link

codecov bot commented Apr 10, 2024

Codecov Report

Attention: Patch coverage is 38.46154% with 16 lines in your changes are missing coverage. Please review.

Project coverage is 77.57%. Comparing base (07f5fe4) to head (a81d56c).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
- Coverage   77.86%   77.57%   -0.29%     
==========================================
  Files          33       33              
  Lines        3560     3586      +26     
  Branches      579      595      +16     
==========================================
+ Hits         2772     2782      +10     
- Misses        788      804      +16     
Files Coverage Δ
...rc/core/dependency_managers/dependency_manager.mjs 100.00% <100.00%> (ø)
...re/dependency_managers/helm_dependency_manager.mjs 90.47% <100.00%> (+0.15%) ⬆️
...dependency_managers/keytool_dependency_manager.mjs 87.65% <100.00%> (+0.15%) ⬆️
src/core/helm.mjs 91.30% <100.00%> (+0.39%) ⬆️
src/core/keytool.mjs 100.00% <100.00%> (ø)
src/commands/init.mjs 75.43% <50.00%> (-0.93%) ⬇️
src/commands/prompts.mjs 68.14% <50.00%> (-0.28%) ⬇️
src/commands/account.mjs 67.66% <33.33%> (-0.53%) ⬇️
src/commands/cluster.mjs 58.01% <0.00%> (-0.90%) ⬇️
src/commands/mirror_node.mjs 0.00% <0.00%> (ø)
... and 3 more

Impacted file tree graph

@JeffreyDallas JeffreyDallas marked this pull request as ready for review April 10, 2024 15:28
@JeffreyDallas JeffreyDallas requested a review from a team as a code owner April 10, 2024 15:28
jeromy-cannon
jeromy-cannon previously approved these changes Apr 11, 2024
…type-check

Signed-off-by: Jeffrey Tang <[email protected]>

# Conflicts:
#	README.md
Copy link
Contributor

Unit Test Results - Linux

  1 files  ±0   17 suites  ±0   1m 29s ⏱️ -12s
114 tests ±0  114 ✅ ±0  0 💤 ±0  0 ❌ ±0 
139 runs  ±0  139 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit a81d56c. ± Comparison against base commit 07f5fe4.

Copy link
Contributor

Unit Test Results - Windows

  1 files  ±0   17 suites  ±0   1m 23s ⏱️ -5s
114 tests ±0  114 ✅ ±0  0 💤 ±0  0 ❌ ±0 
139 runs  ±0  139 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit a81d56c. ± Comparison against base commit 07f5fe4.

@JeffreyDallas JeffreyDallas merged commit d055496 into main Apr 12, 2024
23 of 25 checks passed
@JeffreyDallas JeffreyDallas deleted the 00045-D-type-check branch April 12, 2024 20:32
swirlds-automation added a commit that referenced this pull request Apr 19, 2024
## [0.24.0](v0.23.0...v0.24.0) (2024-04-19)

### Features

* add `solo account create` account alias support ([#227](#227)) ([fe865f8](fe865f8))
* add node refresh subcommand ([#211](#211)) ([d1017d8](d1017d8))
* add node refresh subcommand ([#211](#211)) ([d38918c](d38918c))
* adding some type check ([#217](#217)) ([d055496](d055496))
* allow solo to deploy pre-configured resource profiles ([#175](#175)) ([fef8a48](fef8a48))

### Bug Fixes

* add recursive option to mkdir ([#215](#215)) ([9d9a126](9d9a126))
* address PR comments from [#211](#211) ([#224](#224)) ([6cc237c](6cc237c))
* return chart-path correctly when chart-directory is specified ([#212](#212)) ([90616cc](90616cc))
* update how documented generated ([#216](#216)) ([7d2499e](7d2499e))
@swirlds-automation
Copy link
Contributor

🎉 This PR is included in version 0.24.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@swirlds-automation
Copy link
Contributor

🎉 This PR is included in version 0.24.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Do type check of function parameters
3 participants