feat: set the GDI-spec compliant defs and warn about missing service.name

[rauno56]

• feat: set the defaults on limits
• feat: warn about missing service.name and no instrumentations

Description

Addresses some of the concerns from GDI spec review.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Internal change (a change which is not visible to the package consumers)
• Added automated tests
• Unit tests have been added/updated
• Documentation has been updated
• Change file has been generated (npm run change:new)
• Delete this branch (after the PR is merged)

[codecov-commenter]

Codecov Report

Merging #344 (ad33e44) into main (0b7a6d9) will increase coverage by 0.15%.
The diff coverage is 100.00%.

❗ Current head ad33e44 differs from pull request most recent head 1f08bae. Consider uploading reports for the commit 1f08bae to get more accurate results

@@            Coverage Diff             @@
##             main     #344      +/-   ##
==========================================
+ Coverage   93.89%   94.05%   +0.15%     
==========================================
  Files          10       10              
  Lines         344      353       +9     
  Branches       89       93       +4     
==========================================
+ Hits          323      332       +9     
  Misses         21       21              

Impacted Files	Coverage Δ
src/options.ts	99.02% <100.00%> (+0.09%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0b7a6d9...1f08bae. Read the comment docs.

[seemk]

Is diag.warn visible in the terminal as well?

[rauno56]

Only if it's enabled.

[rauno56]

I'll merge it, @seemk, but any comments are still welcome!

[owais]

Mistake in docs. Looks good otherwise.