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

Refactoring of the Source Code Generator #1141

Merged
merged 3 commits into from
Feb 3, 2024
Merged

Refactoring of the Source Code Generator #1141

merged 3 commits into from
Feb 3, 2024

Conversation

paulushub
Copy link
Contributor

@paulushub paulushub commented Feb 3, 2024

Description

  • Moved the Svg_Model.cs file as SvgModel.cs into the main source.
  • Restore the standard naming of the generated code files with *.g.cs suffix.
  • Set the output directory of the generated codes to Generated.
  • The generated source codes can now be viewed when you turn on Show All Files.
  • Added the Generated directory to the main source codes.
  • Moved the Svg.Custom project to the Tests directory.

Type of change

  • Refactoring of generator code, no change in main source codes (non-breaking change which fixes an issue)
  • This change does not require a documentation update
  • SvgModel.cs, originally part of the code generator, includes attribute classes.
  • Resolved review issues.

How Has This Been Tested?

  • SvgW3CTestRunner is tested for each supported platform
  • Svg.UnitTests tested on command line for each supported platform
  • Svg.Benchmarks tested on command line for each supported platform.

- Moved the Svg_Model.cs file as SvgModel.cs into the main source.
- Restore the standard naming of the generated code files with "*.g.cs" suffix.
- Set the output directory of the generated codes to "Generated".
- Added the Generated directory to the main source codes.
- Moved the Svg.Custom project to Tests directory.
- NuGet/setup-nuget@v1 to NuGet/setup-nuget@v2
- Needed to eliminate the warning: Node.js 16 actions are deprecated.
Removed the comment out portions.
@paulushub paulushub merged commit a2c69b7 into svg-net:master Feb 3, 2024
8 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 3, 2024
…rators README.md Samples Source Tests doc docfx.json index.md license.txt Refactoring of the Source Code Generator - Moved the Svg_Model.cs file as SvgModel.cs into the main source. - Restore the standard naming of the generated code files with "*.g.cs" suffix. - Set the output directory of the generated codes to "Generated". - Added the Generated directory to the main source codes. - Moved the Svg.Custom project to Tests directory. CONTRIBUTING.md Generators README.md Samples Source Tests doc docfx.json index.md license.txt Update runtests.yml - NuGet/setup-nuget@v1 to NuGet/setup-nuget@v2 - Needed to eliminate the warning: Node.js 16 actions are deprecated. CONTRIBUTING.md Generators README.md Samples Source Tests doc docfx.json index.md license.txt Update AvailableElementsGenerator.cs Removed the comment out portions.
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