In performBump, the versions in packageInfos have already been bumped, including when the prebump hook is called:
|
await callHook(options.hooks?.prebump, modifiedPackages, packageInfos, options.concurrency); |
This contradicts the docs for prebump's version param (though this may be why the full packageInfos param was omitted):
|
* @param name The name of the package as defined in package.json |
|
* @param version The pre-bump version of the package to be published |
|
*/ |
|
prebump?: (packagePath: string, name: string, version: string) => void | Promise<void>; |
This behavior is definitely wrong but could be a little risky to fix outside a major version, since people may be assuming the incorrect behavior.
In
performBump, the versions inpackageInfoshave already been bumped, including when theprebumphook is called:beachball/src/bump/performBump.ts
Line 16 in f0f3f64
This contradicts the docs for
prebump'sversionparam (though this may be why the fullpackageInfosparam was omitted):beachball/src/types/BeachballOptions.ts
Lines 343 to 346 in f0f3f64
This behavior is definitely wrong but could be a little risky to fix outside a major version, since people may be assuming the incorrect behavior.