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

immutability: redesign api to improve verbosity #40

Merged
merged 14 commits into from
Oct 20, 2017
Merged

Conversation

awnumar
Copy link
Owner

@awnumar awnumar commented Oct 16, 2017

No description provided.

Awn added 5 commits September 24, 2017 01:28
Go 1.9 no longer includes the vendor directory in wildcard commands, so we can remove the verbosity.
The example for the New function is pretty much redundant.
Related to #37. More refactoring needed.
@awnumar awnumar added the todo label Oct 16, 2017
@awnumar awnumar mentioned this pull request Oct 16, 2017
Awn added 5 commits October 20, 2017 17:16
This change changes the nomenclature and API used to improve verbosity and encourage better programming style.
The new name is more clear about its intentions.
Use internal API to prevent mutex conflicts.
@awnumar awnumar removed the todo label Oct 20, 2017
@awnumar awnumar merged commit 3f5a358 into development Oct 20, 2017
@awnumar awnumar deleted the immutability branch October 20, 2017 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant