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

Refactoring to meet requirements for 2.x #6

Merged
merged 8 commits into from
Dec 18, 2018
Merged

Conversation

cardil
Copy link
Member

@cardil cardil commented Oct 30, 2018

It's about time for major refactoring of Eid library. Introducing 2.x release with:

  • Better structure
  • More obvious exceptions constructors
  • Better configuration
  • Optional validation

@cardil cardil force-pushed the feature/refactor-for-2.x branch 3 times, most recently from 61c644a to 6e97831 Compare October 30, 2018 00:28
@cardil cardil added this to the v2.0.0 milestone Oct 31, 2018
@cardil cardil self-assigned this Oct 31, 2018
@cardil cardil force-pushed the feature/refactor-for-2.x branch 6 times, most recently from 0849df7 to 0db12f1 Compare November 6, 2018 23:34
It's about time for major refactoring of Eid library. Introducing 2.x
relase with:

 * Better structure
 * More obvious exceptions constructors
 * Better configuration
 * Optional validation
@cardil cardil force-pushed the feature/refactor-for-2.x branch 2 times, most recently from e6a2efa to e515c65 Compare November 15, 2018 21:33
Using properly Double-checked locking pattern for Lazy and
SerializableLazy classes. Additionally, adding more test to
cover the introduced code.
@cardil cardil requested a review from a team December 18, 2018 20:14
@cardil cardil merged commit 86952e7 into develop Dec 18, 2018
@cardil cardil deleted the feature/refactor-for-2.x branch December 18, 2018 20:18
@cardil cardil removed the request for review from a team December 18, 2018 20:18
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