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

Update docs on trimming unused schemas #213

Merged
merged 4 commits into from
Nov 6, 2023
Merged

Update docs on trimming unused schemas #213

merged 4 commits into from
Nov 6, 2023

Conversation

christianhelle
Copy link
Owner

The changes here align the README files and update the DocFX articles with basic information about trimming unused schemas, a feature that was introduced by @kirides in #199

…on .refitter file format

The documentation is updated to include two new options, 'trimUnusedSchema' and 'keepSchemaPatterns', which are used for OpenAPI spec schema pruning. These changes provide users greater control over the generated code, especially when working with large schema definitions.
Extended the README.md documentation to include the new command-line options --trim-unused-schema and --keep-schema in order to provide instructions on how to minimize the generated output and selectively preserve certain schema. These options contribute to more efficient and customizable code generation.
…features

Updated README files for Refitter.SourceGenerator, the main folder, and Refitter subfolder. Removed a redundant "namespace" field. Introduced two new optional fields, "trimUnusedSchema" and "keepSchemaPatterns", which enable more granular control over schema retention when generating code. The first removes unreferenced components schema to minimize the output, while the second uses regular expressions to specify which schema to keep even when "trimUnusedSchema" is true.
Extended the configuration in Program.cs by added more examples that demonstrate how to trim unused schemas and keep certain schemas. These examples are intended to improve user understanding of these features.
@christianhelle christianhelle added documentation Improvements or additions to documentation enhancement New feature, bug fix, or request labels Nov 6, 2023
@christianhelle christianhelle self-assigned this Nov 6, 2023
Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #213 (afd4feb) into main (41e095d) will not change coverage.
Report is 4 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #213   +/-   ##
=======================================
  Coverage   98.30%   98.30%           
=======================================
  Files          55       55           
  Lines        2002     2002           
=======================================
  Hits         1968     1968           
  Misses         17       17           
  Partials       17       17           
Flag Coverage Δ
unittests 98.30% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Copy link

sonarqubecloud bot commented Nov 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

No Coverage information No Coverage information
0.0% 0.0% Duplication

@christianhelle christianhelle merged commit ce1123a into main Nov 6, 2023
847 checks passed
@christianhelle christianhelle deleted the docs branch November 6, 2023 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature, bug fix, or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant