-
Notifications
You must be signed in to change notification settings - Fork 277
Adds in process executable version attribute #2655 #2673
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
base: main
Are you sure you want to change the base?
Adds in process executable version attribute #2655 #2673
Conversation
e6d1f92 to
725b5db
Compare
|
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
|
Still looking for further feedback on naming of the attribute as it ties into defining the entity into the best way. |
|
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
2641540 to
ad77405
Compare
|
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
ad77405 to
cc13076
Compare
|
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
cc13076 to
5c7459f
Compare
5c7459f to
2d6b235
Compare
Progresses #2655
Extension to #2657 and would be added to the entity via second PR to be merged.
Changes
This takes the approach of introducing a version attribute for the executable to enable the entity to be version specific.
Alternatives names would be:
The implemented option was chosen to make it complementary to other instances of name/version tuple.
Note: if the PR is touching an area that is not listed in the existing areas, or the area does not have sufficient domain experts coverage, the PR might be tagged as experts needed and move slowly until experts are identified.
Merge requirement checklist
[chore]