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.

Sign up for GitHub

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

Jump to bottom

BCF-3048: Empty implementation of QueryKey for ChainReader #695

Closed
wants to merge 2 commits into from
Closed

BCF-3048: Empty implementation of QueryKey for ChainReader #695

wants to merge 2 commits into from

Conversation

EasterTheBunny
Copy link
Contributor

@EasterTheBunny EasterTheBunny commented May 7, 2024 •
edited by cedric-cordenier
Loading

core ref: a0d1ce5e9cddc540bba8eb193865646cf0ebc0a8

@EasterTheBunny EasterTheBunny requested a review from a team as a code owner May 7, 2024 16:42
ilija42
ilija42 previously approved these changes May 7, 2024
View reviewed changes
@EasterTheBunny EasterTheBunny requested a review from a team as a code owner May 7, 2024 17:38
jmank88
jmank88 previously approved these changes May 7, 2024
View reviewed changes
augustbleeds
augustbleeds previously approved these changes May 7, 2024
View reviewed changes
@cedric-cordenier
Copy link
Contributor

@EasterTheBunny this PR should be covered by this one: 3a7c0d9

@EasterTheBunny EasterTheBunny force-pushed the BCF-048-implement-empty-query-key branch from 96e0ac1 to 37b7e50 Compare May 8, 2024 14:37
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@EasterTheBunny EasterTheBunny deleted the BCF-048-implement-empty-query-key branch May 8, 2024 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilija42 ilija42 ilija42 left review comments

@augustbleeds augustbleeds augustbleeds left review comments

@jmank88 jmank88 jmank88 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@EasterTheBunny @cedric-cordenier @jmank88 @augustbleeds @ilija42