Open-Source-Orchard-Core-Extensions
Open-Source-Orchard-Core-Extensions copied to clipboard
Lombiq
Make NuGetTest updates better (OSOE-816)
This is not a fully planned out suggestion, more of a problem statement and opener to think together.
When you change an OSOCE project today, you don't have to publish an alpha NuGet package. This is OK, since not every change warrants an alpha due to it being used in another NuGet solution (or things like docs or sample changes don't need a NuGet release ever).
However, this results in versions used in the NuGetTest package gradually going out of date. That is, until an OC upgrade requires an update to every project, when all potential breaking changes will arrive at once, and you have to fix the NuGetTest solution. Or a project's NuGet publish may fail as well, e.g. due to incomplete project-package references, or something breaking due to the Node build.
A couple of ideas on how to tackle this:
- Require a NuGet release before approving every submodule PR, if the changes in that PR affect the NuGet package (excluding e.g. the mentioned samples or docs changes). However, this is a chore.
- Run a NuGet package creation, but without publishing to NuGet.org, for every submodule commit, kind of like a CI build. This would at least prevent introducing NuGet publishing errors.
Related issues that might help here: https://github.com/Lombiq/Open-Source-Orchard-Core-Extensions/issues/703 and https://github.com/Lombiq/GitHub-Actions/issues/70.
