Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

Variadic arguments in core.NewNamespace #885

Merged
merged 1 commit into from
Apr 28, 2016

Conversation

marcin-krolik
Copy link
Collaborator

Fixes #883 and #884

Summary of changes:

  • Changed signature of core.NewNamespace to variadic arguments
  • Introduced new function to add many static elements
  • Applied changes to unit tests

Testing done:

  • all unit tests

@intelsdi-x/snap-maintainers

- Introduced new function to add many static elements
- Applied changes to unit tests
@marcin-krolik marcin-krolik changed the title - Changed signature of core.NewNamespace to variadic arguments Variadic arguments in core.NewNamespace Apr 28, 2016
@jcooklin
Copy link
Collaborator

LGTM 🚢 it.

@tiffanyfay
Copy link
Contributor

👍 This is definitely cleaner.

@tiffanyfay tiffanyfay merged commit 891c09f into intelsdi-x:master Apr 28, 2016
@marcin-krolik marcin-krolik deleted the kromar-sig-fix branch August 29, 2016 12:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants