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

refact: typo atractive -> attractive #290

Merged
merged 1 commit into from
Jan 13, 2022

Conversation

fannemel
Copy link
Contributor

@fannemel fannemel commented Jan 13, 2022

A lot of public classes and methods are changed so if this change is implemented, the version should be increased to notify there is no backwards compatability. An option might be to introduce copies and mark the ones with typos as deprecated first.

@EvenSol
Copy link
Collaborator

EvenSol commented Jan 13, 2022

I agree that we make a new version. I don't think backwards combability is a big problem at the moment (the backwards combability on serialized objects have been broken in earlier releases also).

@EvenSol EvenSol added this to the v.2.3.0 milestone Jan 13, 2022
@EvenSol EvenSol merged commit 185bf09 into equinor:master Jan 13, 2022
@fannemel fannemel deleted the typo_attractive branch January 14, 2022 17:44
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.

2 participants