Skip to content

Fix/norms mapping #1726

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jan 14, 2016
Merged

Fix/norms mapping #1726

merged 2 commits into from
Jan 14, 2016

Conversation

gmarz
Copy link
Contributor

@gmarz gmarz commented Jan 14, 2016

  • Refactoring of norms:
    • NormsMapping renamed to Norms
    • Implemented NormsDescriptor - previously Norms() took an object directly.
  • Added a way to test only options that can be mapped via the fluent API
  • Attribute types now live under their respective type folder instead of all being bunched under a single folder.

@Mpdreamz
Copy link
Member

LGM 👍

Mpdreamz added a commit that referenced this pull request Jan 14, 2016
@Mpdreamz
Merge pull request #1726 from elastic/fix/norms-mapping
68166c3 
Fix/norms mapping
@Mpdreamz Mpdreamz merged commit 68166c3 into master Jan 14, 2016
@Mpdreamz Mpdreamz deleted the fix/norms-mapping branch January 14, 2016 16:23
@gmarz gmarz added this to the 2.0.0-alpha2 milestone Jan 15, 2016
@gmarz gmarz added the Bug label Jan 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.0.0-alpha2
Development

Successfully merging this pull request may close these issues.
2 participants
@gmarz @Mpdreamz