search

elastic / ecs-logging

ECS Logging - Common resources and issues for the language specific ECS loggers
https://www.elastic.co/guide/en/ecs-logging/overview/master/intro.html
Apache License 2.0
41 stars 15 forks source link

Fix links in "synchronize ecs-logging spec" PR descriptions #64

Closed trentm closed 3 years ago

trentm commented 3 years ago

Currently links in the "Changesets" section of sync PR descriptions point to repo "elastic/all" instead of "elastic/ecs-logging".

For example, the PR description for https://github.com/elastic/ecs-logging-nodejs/pull/85 was (I have since manually corrected it):


### What
  ECS logging specs automatic sync

  ### Why
  *Changeset*
* https://github.com/elastic/all/commit/a617a44 Adding service.node.name to the spec (https://github.com/elastic/all/pull/61)

Note that the linked repo is "all". This should be "ecs-logging".

My change also tweaks the Markdown spacing.

apmmachine commented 3 years ago

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

#### Build stats * Start Time: 2021-08-30T17:04:34.290+0000 * Duration: 3 min 30 sec * Commit: 7651624c21f2bddc9bf643c5b19467cf11edf7ed #### Trends :test_tube: ![Image of Build Times](https://apm-ci.elastic.co/job/apm-shared/job/ecs-logging-update-specs-mbp/job/PR-64/buildTimeGraph/png)