opentelemetry-js
opentelemetry-js copied to clipboard
Bring API package back into main repo
During the SIG meeting today we discussed the idea of bringing the API back into the main repo. Originally we split it out in order to ensure we didn't accidentally make breaking changes and to promote stability and intentional changes. Since then, the pain of updating the API to add new methods has proven to be worse than the value we gain by having the API in a separate repository. Creating this issue to get the opinions of those who are not able to regularly attend SIG meetings.
/cc @open-telemetry/javascript-maintainers
Is it planned to include API into the lerna project and release together with SDK/experimental or is it just moved to have it in the same repo but keep release independent of each other?
Is it planned to include API into the lerna project and release together with SDK/experimental or is it just moved to have it in the same repo but keep release independent of each other?
AFAICT, we are releasing experimental packages separately from the stable packages, with different versions. Though, the experimental packages are typically released simultaneously with the stable packages.
I'm fine with that, i would say that it may be worth to add it into the lerna project but in a dedicated top-level directory to make a difference with sdks components ?
Is it planned to include API into the lerna project and release together with SDK/experimental or is it just moved to have it in the same repo but keep release independent of each other?
AFAICT, we are releasing experimental packages separately from the stable packages, with different versions. Though, the experimental packages are typically released simultaneously with the stable packages.
We are actually releasing everything together but with different versions.
Dedicated top-level dir seems ok to me if that's how we want to go. Maybe we rename packages/ to sdk/?
I definitely want it in the lerna repo since that is the main advantage of moving it.
I think we should avoid releasing API if there are no changes. But I guess this can be archived also here as it's done also in contrib.
This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days.
API package has been moved in the main repo. Closing as done. Please feel free to reopen if there is anything left to be done.