Skip to content

Commit

Permalink
feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
lmolkova committed Jan 11, 2025
1 parent d3da146 commit d09cce3
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ requirements and recommendations.
- [Sign the CLA](#sign-the-cla)
- [How to Contribute](#how-to-contribute)
- [Which semantic conventions belong in this repo](#which-semantic-conventions-belong-in-this-repo)
- [Suggesting conventions for a new areas](#suggesting-conventions-for-a-new-areas)
- [Suggesting conventions for a new area](#suggesting-conventions-for-a-new-area)
- [Prerequisites](#prerequisites)
- [1. Modify the YAML model](#1-modify-the-yaml-model)
- [Code structure](#code-structure)
Expand Down
6 changes: 5 additions & 1 deletion docs/general/how-to-define-semantic-conventions.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,10 @@ for new areas or make substantial changes to the existing ones.

### Best practices

> [!NOTE]
>
> This section contains non-normative guidance.
#### Defining attributes

Reuse existing attributes when possible. Look through [existing conventions](/docs/attributes-registry/) for similar areas,
Expand Down Expand Up @@ -89,7 +93,7 @@ Consider the scope of the attribute and how it may evolve in the future:
For example, if creating a system-specific attribute, evaluate whether other systems
in the same domain might need a similar attribute in the future.

Similarly, instead of defining a simple boolean flag like `foo.is_error`, consider a
Similarly, instead of defining a simple boolean flag indicating a success or failure, consider a
more extensible approach, such as using a `foo.status_code` attribute to include additional details.

- When defining a broad attribute applicable across multiple domains or systems,
Expand Down

0 comments on commit d09cce3

Please sign in to comment.