fix(core): getNumberFromEnv should return number | undefined (open-telemetry#5874)

Author: shubham-vunet

CHANGELOG.md

@@ -36,6 +36,7 @@ For notes on migrating to 2.x / 0.200.x see [the upgrade guide](doc/upgrade-to-2
 * fix(resource): do not trigger `Accessing resource attributes before async attributes settled` warning when detecting resources [#5546](https://github.com/open-telemetry/opentelemetry-js/pull/5546) @dyladan
   * verbose logging of detected resource removed
 * fix(resource): use dynamic import over require to improve ESM compliance [#5298](https://github.com/open-telemetry/opentelemetry-js/pull/5298) @xiaoxiangmoe
+* fix(core): `getNumberFromEnv` should return number | undefined [#5874](https://github.com/open-telemetry/opentelemetry-js/pull/5874) @shubham-vunet
 
 ### :books: Documentation
 

packages/opentelemetry-core/src/platform/browser/environment.ts

@@ -22,7 +22,7 @@ export function getBooleanFromEnv(_: string): boolean | undefined {
   return undefined;
 }
 
-export function getNumberFromEnv(_: string): boolean | undefined {
+export function getNumberFromEnv(_: string): number | undefined {
   return undefined;
 }