Skip to content

[core] Introduce StructureValueAccessor#getFieldAccessor.#560

Open
dmvk wants to merge 2 commits intomasterfrom
field_accessor
Open

[core] Introduce StructureValueAccessor#getFieldAccessor.#560
dmvk wants to merge 2 commits intomasterfrom
field_accessor

Conversation

@dmvk
Copy link
Copy Markdown

@dmvk dmvk commented Jun 8, 2021

No description provided.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Jun 8, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

return result;
}
}
throw new IllegalStateException(
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

NIT: This should be probably IlegalArgumentException

@sonarqubecloud
Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell B 4 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

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.

2 participants