Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: Support boolean additional_properties in JSON schema helper objects #1188

Merged
merged 9 commits into from
Dec 7, 2022

Conversation

edgarrmondragon
Copy link
Collaborator

@edgarrmondragon edgarrmondragon commented Nov 15, 2022

This is a smaller incremental change extracted from #768


📚 Documentation preview 📚: https://meltano-sdk--1188.org.readthedocs.build/en/1188/

@edgarrmondragon edgarrmondragon force-pushed the feat/boolean-additional-properties branch from 5dc8d0a to dff6333 Compare November 15, 2022 16:47
@codecov
Copy link

codecov bot commented Nov 15, 2022

Codecov Report

Merging #1188 (e07d900) into main (9d6a48a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1188   +/-   ##
=======================================
  Coverage   83.67%   83.68%           
=======================================
  Files          42       42           
  Lines        3909     3911    +2     
  Branches      666      667    +1     
=======================================
+ Hits         3271     3273    +2     
  Misses        474      474           
  Partials      164      164           
Impacted Files Coverage Δ
singer_sdk/typing.py 94.94% <100.00%> (+0.05%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@edgarrmondragon edgarrmondragon requested review from kgpayne and removed request for cjohnhanson November 15, 2022 17:15
@edgarrmondragon
Copy link
Collaborator Author

Low risk, quick win. Merging.

@edgarrmondragon edgarrmondragon merged commit 747a202 into main Dec 7, 2022
@edgarrmondragon edgarrmondragon deleted the feat/boolean-additional-properties branch December 7, 2022 02:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants