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

feat: overloaded the function to have a custom query #326

Closed
wants to merge 6 commits into from

Conversation

Abiji-2020
Copy link
Contributor

Overloaded the read function to support the feature.

closes #288
/claim #288

@Abiji-2020
Copy link
Contributor Author

CI is failing due to the differences in the expected.out and the base.out that is created during test. Will Optmise and update the expected base.out.

@Abiji-2020
Copy link
Contributor Author

image

Even though the values are same, it still shows error and in my local the test are passing
image

@Neptune650
Copy link
Contributor

I already submitted a PR for this #322, so I'm not quite sure what's the need for this one.

@Abiji-2020
Copy link
Contributor Author

I already submitted a PR for this #322, so I'm not quite sure what's the need for this one.

Actually i forgot to mention in the issue, so i just pushed and submited

@Neptune650
Copy link
Contributor

This user is known to do plagiarism:
tailcallhq/gh-workflow#12 (comment)

Signed-off-by: Abinand P <abinand0911@gmail.com>
Signed-off-by: Abinand P <abinand0911@gmail.com>
Signed-off-by: Abinand P <abinand0911@gmail.com>
Signed-off-by: Abinand P <abinand0911@gmail.com>
Signed-off-by: Abinand P <abinand0911@gmail.com>
@Abiji-2020 Abiji-2020 force-pushed the feature-to-read-overload branch from 57be236 to a55373b Compare October 21, 2024 00:04
@Abiji-2020
Copy link
Contributor Author

Abiji-2020 commented Oct 21, 2024

This user is known to do plagiarism: tailcallhq/rust-gh-workflows#12 (comment)

That was a issue to be resolved with them, and i have approached here as to overload the function for the read query.

@Abiji-2020
Copy link
Contributor Author

@ChuckHend could you please review it.?

@ChuckHend
Copy link
Member

@ChuckHend could you please review it.?

We are currently reviewing #322 since it was opened first.

@ChuckHend
Copy link
Member

Closing this issue. The feature was merged in #322

@ChuckHend ChuckHend closed this Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature Request: Conditional reading for PGMQ
3 participants