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/fix validate open rpc document example #834

Conversation

codingwithmanny
Copy link

Context:
Proposed solution updates example code with small fixes for imports and adjusted output to handle error a bit clearer.

Working Example:
https://replit.com/@codingwithmanny/open-rpc-validateOpenRPCDocument?v=1#index.ts

codingwithmanny added 3 commits January 2, 2023 23:22
Adjusted example code with import and try/catch

BREAKING CHANGE: N
Removed try catch in favour of result because it doesn't throw an error

BREAKING CHANGE: N
@codecov
Copy link

codecov bot commented Jan 2, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (32a98ff) to head (820f036).

❗ Current head 820f036 differs from pull request most recent head fbe412d. Consider uploading reports for the commit fbe412d to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #834   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           13        13           
  Lines          322       322           
  Branches        74        74           
=========================================
  Hits           322       322           

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

@BelfordZ
Copy link
Member

BelfordZ commented May 7, 2024

sorry it took so long to look at this, but thanks LGTM!

@BelfordZ BelfordZ merged commit cbd7295 into open-rpc:master May 7, 2024
2 checks passed
@BelfordZ
Copy link
Member

BelfordZ commented May 7, 2024

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@codingwithmanny
Copy link
Author

codingwithmanny commented May 7, 2024

sorry it took so long to look at this, but thanks LGTM!

@BelfordZ All good. I figured there is a lot going on.

PS: Next time I'm in Van, let me know if you'd be down to meet up :)

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

Successfully merging this pull request may close these issues.

2 participants