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

docs: gitpod review #563

Merged
merged 3 commits into from
Apr 1, 2024
Merged

Conversation

bguiz
Copy link
Contributor

@bguiz bguiz commented Mar 14, 2024

Description:

What

  • review the gitpod section
  • add sample commands

Why

  • make it clearer, and easier to follow along
  • allow users to quickly interact with it (using sample commands), hello world style

Related issue(s):

Fixes: nil

Notes for reviewer:

Refs:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)
    • Manual testing of the sample command only

@codecov-commenter
Copy link

codecov-commenter commented Mar 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.02%. Comparing base (e2f0a7b) to head (92d3a00).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #563   +/-   ##
=======================================
  Coverage   51.02%   51.02%           
=======================================
  Files          24       24           
  Lines        1172     1172           
  Branches      131      131           
=======================================
  Hits          598      598           
  Misses        550      550           
  Partials       24       24           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bguiz bguiz force-pushed the docs/gitpod-review branch from b027629 to d47d3ee Compare March 14, 2024 07:52
@yaroslav-007
Copy link
Contributor

yaroslav-007 commented Mar 18, 2024

Hello,

@bguiz
Copy link
Contributor Author

bguiz commented Mar 19, 2024

Thanks for the review! I've addressed your comments in cc7d1b0

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link
Contributor

@yaroslav-007 yaroslav-007 left a comment

Choose a reason for hiding this comment

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

LGTM

@georgi-l95 georgi-l95 merged commit a67b7de into hiero-ledger:main Apr 1, 2024
9 of 11 checks passed
andrewb1269hg added a commit that referenced this pull request Oct 29, 2024
commit 336082b Remove db override
commit 3af747a chore: Bump actions/setup-node from 4.0.2 to 4.1.0 (#813)
commit 51a228f Update readme about keys usage (#8)
commit 61470e8 ci: Updated nick-fields/retry v3.2.0 with step-security/retry v3.0.0 (#655)
commit 8379b9f fix: resolve windows issue (#138)
commit 9840fd1 add gitignore file
commit 9cd1a7b Add submodules for mirrornode and point them to services 0.11.0 and mirror 0.27.0
commit a67b7de docs: gitpod review (#563)
commit ae9e705 add .env file for services repositories and versions
commit b927f09 fix spacing in docker compose file
commit ba4b9d4 Add apache license
commit cb12fa8 add readme
commit cf71d97 Add docker-compose override files, configs, and initial address books
commit d343276 Add env variables for various memory limits (#13)
commit d4cf91c bump version (#237)
commit ff6a728 chore: update PR check workflow (#568)

Signed-off-by: Andrew Brandt <andrew.brandt@hashgraph.com>
rbarkerSL added a commit that referenced this pull request Jan 15, 2025
commit 2466b28 chore: Bump codecov/codecov-action from 5.0.4 to 5.0.7 (#860)
commit 336082b Remove db override
commit 3af747a chore: Bump actions/setup-node from 4.0.2 to 4.1.0 (#813)
commit 41f6b53 chore: Bump step-security/harden-runner from 2.10.1 to 2.10.2 (#852)
commit 51a228f Update readme about keys usage (#8)
commit 60ca5dd chore: Bump codecov/codecov-action from 4.6.0 to 5.0.0 (#843)
commit 61470e8 ci: Updated nick-fields/retry v3.2.0 with step-security/retry v3.0.0 (#655)
commit 8379b9f fix: resolve windows issue (#138)
commit 97c6b24 chore: Bump codecov/codecov-action from 5.0.2 to 5.0.4 (#858)
commit 9840fd1 add gitignore file
commit 9cd1a7b Add submodules for mirrornode and point them to services 0.11.0 and mirror 0.27.0
commit a67b7de docs: gitpod review (#563)
commit ae9e705 add .env file for services repositories and versions
commit b927f09 fix spacing in docker compose file
commit ba4b9d4 Add apache license
commit c06ba33 fix: small fix
commit cb12fa8 add readme
commit cf71d97 Add docker-compose override files, configs, and initial address books
commit d343276 Add env variables for various memory limits (#13)
commit d4cf91c bump version (#237)
commit ff6a728 chore: update PR check workflow (#568)

Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants