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

Allow non-string infra defaults for plugins #1921

Merged
merged 7 commits into from
Feb 1, 2024

Conversation

araghukas
Copy link
Contributor

@araghukas araghukas commented Jan 29, 2024

Fix for #1920

  • I have added the tests to cover my changes.
  • I have updated the documentation and CHANGELOG accordingly.
  • I have read the CONTRIBUTING document.

@araghukas araghukas requested a review from a team as a code owner January 29, 2024 23:36
Copy link

codecov bot commented Jan 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8d22e7d) 84.52% compared to head (67b870c) 85.14%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1921      +/-   ##
===========================================
+ Coverage    84.52%   85.14%   +0.61%     
===========================================
  Files          295      181     -114     
  Lines        14488    11293    -3195     
  Branches       195        0     -195     
===========================================
- Hits         12246     9615    -2631     
+ Misses        2108     1678     -430     
+ Partials       134        0     -134     
Flag Coverage Δ *Carryforward flag
Dispatcher 92.53% <ø> (ø) Carriedforward from 52af8a6
Functional_Tests ?
SDK 79.58% <100.00%> (+0.09%) ⬆️
UI_Backend ?
UI_Frontend ?

*This pull request uses carry forward flags. Click here to find out more.

Copy link
Member

@kessler-frost kessler-frost left a comment

Choose a reason for hiding this comment

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

Looks gtg apart from the changelog comment 💯 !

CHANGELOG.md Outdated Show resolved Hide resolved
@kessler-frost kessler-frost merged commit 8598589 into develop Feb 1, 2024
14 checks passed
@kessler-frost kessler-frost deleted the 1920-allow-non-string-infra-defaults branch February 1, 2024 01:29
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