8000 Finalize 1.0.1 changelog by webmat · Pull Request #472 · elastic/ecs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Finalize 1.0.1 changelog #472

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

Merged
merged 3 commits into from
May 24, 2019
Merged

Finalize 1.0.1 changelog #472

merged 3 commits into from
May 24, 2019

Conversation

webmat
Copy link
Contributor
@webmat webmat commented May 24, 2019

No description provided.

@webmat webmat self-assigned this May 24, 2019
@webmat webmat added the v1.0.1 label May 24, 2019
@webmat webmat requested a review from ruflin May 24, 2019 14:58
CHANGELOG.md Outdated

* Fix typo in "process" fieldset definition that prevented the example value for
`process.pid` from being picked displayed. #464, #470

### Added

* Add generated source code for Go. #249
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if in the future most of these entries should go into something like "developer changelog" because it does not affect the schema. Would be good to have a place where only changes about the schema are noted.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well the example now shows up in the doc. I agree it's not a change to the schema per se, though.

Actually, perhaps the changelog entry should be more straightforward, however:

Add example for process.pid and process.ppid

Copy link
Contributor

Choose a reason for hiding this comment

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

My comment was more general and not only for this specific line.

@webmat webmat force-pushed the changelog-1.0.1-part2 branch from 3ec5415 to d905ed6 Compare May 24, 2019 17:11
@webmat webmat merged commit ab5e966 into elastic:1.0 May 24, 2019
webmat pushed a commit to webmat/ecs that referenced this pull request May 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0