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

Improve Koa and KoaJsonRpc classes #580

Merged
merged 1 commit into from
Oct 3, 2022

Conversation

georgi-l95
Copy link
Collaborator

@georgi-l95 georgi-l95 commented Oct 3, 2022

Signed-off-by: georgi-l95 glazarov95@gmail.com

Description:
Initialise Koa server in Koa Json RPC class, no need to be separate.

Related issue(s):

Fixes #530

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@georgi-l95 georgi-l95 linked an issue Oct 3, 2022 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Oct 3, 2022

Codecov Report

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

Coverage data is based on head (5ad4c60) compared to base (88f52d9).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #580   +/-   ##
=======================================
  Coverage   72.02%   72.02%           
=======================================
  Files          15       15           
  Lines        1026     1026           
  Branches      166      166           
=======================================
  Hits          739      739           
  Misses        236      236           
  Partials       51       51           

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.

@georgi-l95 georgi-l95 self-assigned this Oct 3, 2022
@georgi-l95 georgi-l95 added enhancement New feature or request limechain labels Oct 3, 2022
@georgi-l95 georgi-l95 added this to the 0.10 milestone Oct 3, 2022
@georgi-l95 georgi-l95 marked this pull request as ready for review October 3, 2022 12:49
Nana-EC
Nana-EC previously approved these changes Oct 3, 2022
Copy link
Collaborator

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG.
Question though. Is there a need to have packages/server/src/server.ts and /Users/nanaessilfie-conduah/Documents/GitHub/hedera-json-rpc-relay/packages/server/src/koaJsonRpc/index.ts be separate.
It's unclear why they need to be separate

packages/server/src/server.ts Show resolved Hide resolved
Signed-off-by: georgi-l95 <glazarov95@gmail.com>

fix

Signed-off-by: georgi-l95 <glazarov95@gmail.com>
@georgi-l95 georgi-l95 force-pushed the 530-increase-cohesion-between-koa-and-koajsonrpc branch from 5ad4c60 to b823326 Compare October 3, 2022 14:20
@sonarcloud
Copy link

sonarcloud bot commented Oct 3, 2022

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 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Collaborator

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG

@Nana-EC Nana-EC modified the milestones: 0.10, 0.9.0 Oct 3, 2022
@georgi-l95 georgi-l95 merged commit d791ee4 into main Oct 3, 2022
@georgi-l95 georgi-l95 deleted the 530-increase-cohesion-between-koa-and-koajsonrpc branch October 3, 2022 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request limechain
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Increase cohesion between koa and koaJsonRpc
3 participants