Skip to content

feat(wpm): bump file support object#25

Merged
why520crazy merged 10 commits into
worktile:masterfrom
smile1016:zxl/fix-bumpfile
May 14, 2026
Merged

feat(wpm): bump file support object#25
why520crazy merged 10 commits into
worktile:masterfrom
smile1016:zxl/fix-bumpfile

Conversation

@smile1016
Copy link
Copy Markdown
Contributor

No description provided.

@smile1016 smile1016 closed this May 13, 2026
@smile1016 smile1016 reopened this May 13, 2026
Comment thread src/lifecycles/bump.ts
}

private normalizeBumpFiles(bumpFiles: any, basePath: string): string[] {
private normalizeBumpFiles(bumpFiles: any, basePath: string): BumpFile[] {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这些 normalizeBumpFiles 在 ReleaseHandler 不是已经处理过了吗?在 bump lifecycle 直接使用就好了,为什么还要再次处理一下?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

如果需要把这些逻辑移动到 lifecycle,请把 ReleaseHandler 中的逻辑去除

Comment thread src/lifecycles/select-version.ts Outdated
const tagVersion = await latestSemverTag({ tagPrefix: context.options.tagPrefix });
const version = context.versions.current;
this.logger.info(`current version is ${chalk.green(version)} and latest tag is ${chalk.green(tagVersion)}`);
context.options.releaseAs =context.versions.next
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

加这个是解决什么问题的?

@why520crazy why520crazy merged commit aecc63a into worktile:master May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants