elastic / elastic/ecs-typescript
Not a blocker for this PR, but probably could make our lives easier in the long-run by just attaching a `SPEC_IDENTIFIER` and `GROUP_SPEC_IDENTIFIER` (or similar), which could house everything in the `EcsGroupSpec` interface. Then fewer changes are required if we ever need to pull more data out of the group spec.
Open
enhancement
- Dominant language
- JavaScript
- Stars
- 4
- Forks
- 4
- PR merge metrics
- No merged PRs in 30d
Description
Not a blocker for this PR, but probably could make our lives easier in the long-run by just attaching a `SPEC_IDENTIFIER` and `GROUP_SPEC_IDENTIFIER` (or similar), which could house everything in the `EcsGroupSpec` interface. Then fewer changes are required if we ever need to pull more data out of the group spec.
```ts
type SpecInternals = {
[FIELD_SPEC_IDENTIFIER]?: EcsFieldSpec;
[GROUP_SPEC_IDENTIFIER]?: EcsGroupSpec;
};
```
_Originally posted by @lukeelmers in https://github.com/elastic/ecs-typescript/pull/18#discussion_r1064900259_
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.