Skip to content

Use ".g.cs" extension for all generated files - #284

Merged
Sergio0694 merged 1 commit into
mainfrom
dev/use-g-cs-generated-extension
Jun 1, 2022
Merged

Use ".g.cs" extension for all generated files#284
Sergio0694 merged 1 commit into
mainfrom
dev/use-g-cs-generated-extension

Conversation

@Sergio0694

@Sergio0694 Sergio0694 commented May 30, 2022

Copy link
Copy Markdown
Member

Closes #264

PR Checklist

  • Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
  • Based off latest main branch of toolkit
  • PR doesn't include merge commits (always rebase on top of our main, if needed)
  • Tested code with current supported SDKs
  • New component
    • Pull Request has been submitted to the documentation repository instructions. Link:
    • Added description of major feature to project description for NuGet package (4000 total character limit, so don't push entire description over that)
  • Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Header has been added to all new source files (run build/UpdateHeaders.bat)
  • Contains NO breaking changes
  • Every new API (including internal ones) has full XML docs
  • Code follows all style conventions

@Sergio0694 Sergio0694 added improvements ✨ Improvements to an existing functionality next preview ✈️ This changes will be available in the upcoming preview mvvm-toolkit 🧰 Issues/PRs for the MVVM Toolkit labels May 30, 2022
@will-jj will-jj mentioned this pull request May 30, 2022
Closed
@Sergio0694
Sergio0694 merged commit 77b39b6 into main Jun 1, 2022
@delete-merged-branch
delete-merged-branch Bot deleted the dev/use-g-cs-generated-extension branch June 1, 2022 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Arlodotexe Arlodotexe Arlodotexe approved these changes

Assignees

No one assigned

Labels

improvements ✨ Improvements to an existing functionality mvvm-toolkit 🧰 Issues/PRs for the MVVM Toolkit next preview ✈️ This changes will be available in the upcoming preview

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Coverlet Cannot Cover Module Using MVVM Toolkit

2 participants

@Sergio0694 @Arlodotexe