-
Notifications
You must be signed in to change notification settings - Fork 77
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: add support for workspace context in charts #649
Conversation
end | ||
end | ||
|
||
def self.inject_context_in_value_custom(value) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Method inject_context_in_value_custom
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
ca66841
to
fa87532
Compare
Code Climate has analyzed commit fa87532 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 98.7% (55% is the threshold). This pull request will bring the total coverage in the repository to 50.4%. View more on Code Climate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
🎉 This PR is included in version 9.2.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
# [9.2.0](v9.1.10...v9.2.0) (2024-02-29) ### Features * add support for workspace context in charts ([#649](#649)) ([3e98864](3e98864))
# [9.2.0](ForestAdmin/forest-rails@v9.1.10...v9.2.0) (2024-02-29) ### Features * add support for workspace context in charts ([#649](ForestAdmin/forest-rails#649)) ([3e98864](ForestAdmin/forest-rails@3e98864))
# [9.2.0](ForestAdmin/forest-rails@v9.1.10...v9.2.0) (2024-02-29) ### Features * add support for workspace context in charts ([#649](ForestAdmin/forest-rails#649)) ([3e98864](ForestAdmin/forest-rails@3e98864))
Definition of Done
General
Security