chore(resource-detector-instana): use exported strings for attributes

[maryliag]

Which problem is this PR solving?

• Part Of Update usage of semantic conventions to new exported strings #2025

Short description of the changes

On package opentelemetry-resource-detector-instana:

• Update @opentelemetry/semantic-conventions from ^1.0.0 to ^1.22.0
• Use exported strings for Semantic Resource Attributes.

[JamieDanielson]

Just a note, I recently updated the CONTRIBUTING doc because we no longer need to update the changelog on our own in PRs - it gets updated in the releasing step!

[codecov]

Codecov Report

Merging #2051 (e079ce8) into main (dfb2dff) will decrease coverage by 0.16%.
Report is 37 commits behind head on main.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2051      +/-   ##
==========================================
- Coverage   90.97%   90.82%   -0.16%     
==========================================
  Files         146      148       +2     
  Lines        7492     7672     +180     
  Branches     1502     1537      +35     
==========================================
+ Hits         6816     6968     +152     
- Misses        676      704      +28     

Files	Coverage Δ
...ctor-instana/src/detectors/InstanaAgentDetector.ts	96.42% <100.00%> (ø)

... and 8 files with indirect coverage changes

[maryliag]

Updated to remove the changes on changelog

[trentm]

Just the one README nit.

[trentm]

Perhaps update the import { SemanticResourceAttributes } from "@opentelemetry/semantic-conventions"; a couple lines up.

[maryliag]

Great catch! Done

[JamieDanielson]

Thanks for this @maryliag ! Just a note, it's helpful for reviewers if you actually do not force-push changes, and instead keep the history. Not a big deal on smaller PRs, but as changes are made it can be difficult to see what's changed.