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

chore: fix typos #2134

Closed
wants to merge 22 commits into from
Closed

chore: fix typos #2134

wants to merge 22 commits into from

Conversation

omahs
Copy link
Contributor

@omahs omahs commented Oct 18, 2023

fix typos

Copy link
Member

@vpavlin vpavlin left a comment

Choose a reason for hiding this comment

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

Amazing! Thank you so much!

@alrevuelta alrevuelta self-requested a review October 25, 2023 10:49
@alrevuelta
Copy link
Contributor

@vpavlin build-docker-image ci is failing. doesnt seem related, any idea?

Copy link
Contributor

@alrevuelta alrevuelta left a comment

Choose a reason for hiding this comment

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

lgtm.

@Ivansete-status
Copy link
Collaborator

Hey @omahs !
Thanks so much indeed for the contribution!
When you have some spare time, kindly perform Squash and merge so that your changes can be applied 🥳

omahs and others added 13 commits November 21, 2023 20:28
* feat: amending computeDigest func. + related test cases

* minor fixes

* minor fixes v1: testcase saga continues

---------

Co-authored-by: Vaclav Pavlin <[email protected]>
* libwaku: Avoid global variable and changing callback signature

* Better signature for the callback. Two new parameters have been added:
  one aimed to allow passing the caller result code; the other
  param is to pass an optional userData pointer that might need
  to be linked locally with the Context object. For example, this is needed
  in Rust to make the passed closures live as
  long as the Context.

* waku_example.c: adaptation to the latest changes

* libwaku.h: removing 'waku_set_user_data' function

* libwaku.nim: renaming parameter in WakuCallBack (isOk -> callerRet)
* feat: messageHash attaribute added in SQLite + testcase

* Update tests/waku_archive/test_driver_sqlite_query.nim

Co-authored-by: Ivan FB <[email protected]>

---------

Co-authored-by: Ivan FB <[email protected]>
@omahs
Copy link
Contributor Author

omahs commented Nov 21, 2023

@Ivansete-status Hey, sorry I messed up my branch. I opened another squashed PR here: #2232

@omahs omahs closed this Nov 21, 2023
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.

6 participants