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

Add setting to ignore default value checker #177

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dylan-smith
Copy link

@dylan-smith dylan-smith commented Sep 5, 2024

Fixes #121

Add a setting to ignore the default value checker

  • Add a new boolean property IgnoreDefaultValueChecker to the GenFu class in src/GenFu/GenFu.cs.
  • Modify the New method in the GenFu class to check the IgnoreDefaultValueChecker property before using the DefaultValueChecker.
  • Add a method to set the IgnoreDefaultValueChecker property in the GenFuConfigurator class in src/GenFu/GenFuConfigurator.cs.
  • Add tests in tests/GenFu.Tests/GenFuTests.cs to verify the new behavior of ignoring the default value checker.

For more details, open the Copilot Workspace session.

Fixes MisterJames#121

Add a setting to ignore the default value checker

* Add a new boolean property `IgnoreDefaultValueChecker` to the `GenFu` class in `src/GenFu/GenFu.cs`.
* Modify the `New` method in the `GenFu` class to check the `IgnoreDefaultValueChecker` property before using the `DefaultValueChecker`.
* Add a method to set the `IgnoreDefaultValueChecker` property in the `GenFuConfigurator` class in `src/GenFu/GenFuConfigurator.cs`.
* Add tests in `tests/GenFu.Tests/GenFuTests.cs` to verify the new behavior of ignoring the default value checker.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/MisterJames/GenFu/issues/121?shareId=XXXX-XXXX-XXXX-XXXX).
…ator class

* Add XML comments for the IgnoreDefaultValueChecker method
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.

Add setting to ignore default value checker
1 participant