-
Notifications
You must be signed in to change notification settings - Fork 2
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
Rename 'SvgRenderingTemplate2023' type to 'SvgRenderingTemplate' #4
Comments
I'm generally OK with moving away from years, but versioning often matters, even apart from "freshness". Are we certain that |
If we're going to version these, the year approach doesn't seem too terrible. |
As long as we're reasonably certain there will never be two versions in one year with a breaking change between them. |
I don't think the intent with the year suffix was to process with semantic versioning code. Mostly was to give people a general era and sequence idea, but I forget the full reasoning we had years ago. The hope was these types of things change rarely, but there's no harm in adding a 2023v2, 2023B, or similar as needed. |
To me, the likelihood of there being two different templates within a year (or even a decade), with the same name, is very low. If one really needs to do a breaking change, just name it something else. |
@dmitrizagidulin |
lol YES! Much better! |
I strongly feel that we should drop the year from the template type. (it's not a crypto suite, there's no need to denote "freshness" in the type, and we're moving away from years in crypto suites too).
The text was updated successfully, but these errors were encountered: