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

V2 #308

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

V2 #308

wants to merge 5 commits into from

Conversation

dbatten5
Copy link
Owner

@dbatten5 dbatten5 commented Apr 7, 2024

Introduces some breaking changes:

  • ProjectConfig has been removed in favour of a new UserConfig. Objects of this class behave like a dict, removing the need for the get_option method as config values can be retrieved as if they're dict values. Also removes the need for a to_dict method.
  • The dependecy on pydantic is removed for the schema validation. For validation you now just need to supply a class that implements a dict method, such as a pydantic.BaseModel.

This replaces the `ProjectConfig` class to be used as the main API.
The new interface should be simpler to use and more intuitive as it
behaves like a dict object.
The `ProjectConfig` class is no longer used, as is the `ConfigSchema`
class
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.

None yet

1 participant