Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add fields for Operating System data #5

Merged
merged 1 commit into from
Jun 6, 2018

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented May 29, 2018

There were some previous discussions around the version field and if it should split up into major, minor, bugfix, patch. I chose to only go with version based on the assumption that if someone searches for a specific minor version he will know the major version. So searching for it can be done with a prefix query based on version keyword.

@ruflin ruflin added the review label May 29, 2018
@ruflin ruflin changed the title Add fields for Operating Systemd data Add fields for Operating System data May 29, 2018
Copy link
Member

@andrewkroh andrewkroh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. It needs a rebase due to a conflict.

There were some previous discussions around the version field and if it should split up into major, minor, bugfix, patch. I chose to only go with `version` based on the assumption that if someone searches for a specific minor version he will know the major version. So searching for it can be done with a prefix query based on version keyword.
@ruflin
Copy link
Contributor Author

ruflin commented Jun 6, 2018

@andrewkroh Rebased and #15 opened to reduce conflicts in the CHANGELOG.

@andrewkroh andrewkroh merged commit e17097a into elastic:master Jun 6, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants