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

Add missing step names. #419

Merged
merged 1 commit into from
Sep 10, 2023
Merged

Conversation

shachlanAmazon
Copy link
Contributor

I wasted a lot of time by trying to debug issues in the wrong step. Now all steps will be properly named, and easy to understand where failures happen.


- uses: ./.github/workflows/lint-rust
with:
cargo-toml-folder: ./benchmarks/rust
name: lintbenchmark
Copy link
Collaborator

Choose a reason for hiding this comment

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

lint benchmark

Copy link
Collaborator

@barshaul barshaul left a comment

Choose a reason for hiding this comment

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

FSM

I wasted a lot of time by trying to debug issues in the wrong step.
Now all steps will be properly named, and easy to understand where
failures happen.
@shachlanAmazon shachlanAmazon merged commit 20d87bf into valkey-io:main Sep 10, 2023
@shachlanAmazon shachlanAmazon deleted the rule-names branch September 11, 2023 09:24
Yury-Fridlyand added a commit that referenced this pull request Sep 10, 2024
)

* returning a record replacement with glide strings.
* Node: Add binary variant to `PUBSUB NUMSUB` and `PUBSUB SHARDNUMSUB` (#419)
* Updated PubSub shardnumsub and numsub commands
* Fix tests.
* Add binary variant to PUBSUB subscribe commands
* Fix processing cluster response.

Signed-off-by: Yury-Fridlyand <[email protected]>
Signed-off-by: Yi-Pin Chen <[email protected]>
Co-authored-by: Andrew Carbonetto <[email protected]>
Co-authored-by: Yi-Pin Chen <[email protected]>
janhavigupta007 pushed a commit to janhavigupta007/glide-for-redis that referenced this pull request Sep 11, 2024
…lkey-io#2207)

* returning a record replacement with glide strings.
* Node: Add binary variant to `PUBSUB NUMSUB` and `PUBSUB SHARDNUMSUB` (valkey-io#419)
* Updated PubSub shardnumsub and numsub commands
* Fix tests.
* Add binary variant to PUBSUB subscribe commands
* Fix processing cluster response.

Signed-off-by: Yury-Fridlyand <[email protected]>
Signed-off-by: Yi-Pin Chen <[email protected]>
Co-authored-by: Andrew Carbonetto <[email protected]>
Co-authored-by: Yi-Pin Chen <[email protected]>
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.

2 participants