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

Recommendation: Remove Final #4

Closed
aidan-casey opened this issue Dec 7, 2020 · 1 comment
Closed

Recommendation: Remove Final #4

aidan-casey opened this issue Dec 7, 2020 · 1 comment

Comments

@aidan-casey
Copy link

Hey there! Just wanted to make a quick recommendation to remove the final keyword from the class. It would be very helpful to be able to extend the registry.

For example, consider a configuration management class. This could extend the registry class and provide a load method which would populate the registry with configured values from a JSON file, etc.

Willing to make a pull request if this is functionality you would be willing to consider!

@Awilum
Copy link
Member

Awilum commented Feb 20, 2021

removed in v3.0.0

@Awilum Awilum closed this as completed Feb 20, 2021
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

No branches or pull requests

2 participants