Skip to content

feat(bindings/nodejs): add retry layer#3484

Merged
Xuanwo merged 10 commits intomainfrom
nodejs-layer
Nov 14, 2023
Merged

feat(bindings/nodejs): add retry layer#3484
Xuanwo merged 10 commits intomainfrom
nodejs-layer

Conversation

@suyanhanx
Copy link
Copy Markdown
Member

No description provided.

@github-actions github-actions Bot added the releases-note/feat The PR implements a new feature or has a title that begins with "feat" label Nov 4, 2023
@suyanhanx
Copy link
Copy Markdown
Member Author

It looks like irrelevant tests were triggered.

Comment thread .github/workflows/bindings_nodejs.yml
Comment thread bindings/nodejs/generated.js
Comment thread bindings/nodejs/tests/suites/index.mjs Outdated
@suyanhanx suyanhanx force-pushed the nodejs-layer branch 2 times, most recently from a65e8e2 to 3d002e5 Compare November 9, 2023 09:36
@suyanhanx suyanhanx marked this pull request as ready for review November 9, 2023 09:36
@suyanhanx suyanhanx requested a review from PsiACE as a code owner November 9, 2023 09:36
Comment thread bindings/nodejs/src/lib.rs Outdated
Comment thread bindings/nodejs/src/lib.rs Outdated
Comment thread bindings/nodejs/src/lib.rs Outdated
Comment thread bindings/nodejs/src/lib.rs Outdated
Comment thread bindings/nodejs/index.js
@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Nov 11, 2023

Hi, I don't like the changes in this PR, but I can't have better ideas withou playing with it. Can I have a try based on this PR, maybe push some changes if it's ok?

@suyanhanx
Copy link
Copy Markdown
Member Author

Hi, I don't like the changes in this PR, but I can't have better ideas withou playing with it. Can I have a try based on this PR, maybe push some changes if it's ok?

It's ok. It's up to you.😇

@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Nov 13, 2023

Hi, @suyanhanx, here's my idea but I'm not well-versed in nodejs. Could you take it further and delve deeper into it?

@suyanhanx
Copy link
Copy Markdown
Member Author

Hi, @suyanhanx, here's my idea but I'm not well-versed in nodejs. Could you take it further and delve deeper into it?

I see. I'll finish this.

@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Nov 13, 2023

I see. I'll finish this.

Thanks a lot!

@suyanhanx suyanhanx requested a review from Xuanwo November 13, 2023 15:25
Comment thread .github/workflows/bindings_nodejs.yml
Comment thread bindings/nodejs/src/lib.rs Outdated
suyanhanx and others added 9 commits November 14, 2023 12:32
Signed-off-by: suyanhanx <suyanhanx@gmail.com>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: suyanhanx <suyanhanx@gmail.com>
Signed-off-by: suyanhanx <suyanhanx@gmail.com>
Signed-off-by: suyanhanx <suyanhanx@gmail.com>
Signed-off-by: suyanhanx <suyanhanx@gmail.com>
Signed-off-by: suyanhanx <suyanhanx@gmail.com>
Signed-off-by: suyanhanx <suyanhanx@gmail.com>
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks a lot for your effort!

@Xuanwo Xuanwo merged commit 3cd08b6 into main Nov 14, 2023
@Xuanwo Xuanwo deleted the nodejs-layer branch November 14, 2023 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants