Skip to content

Fix the problem of theme version compatibility#1705

Merged
ruibaby merged 1 commit intohalo-dev:masterfrom
JohnNiang:bug/version-comparison
Mar 5, 2022
Merged

Fix the problem of theme version compatibility#1705
ruibaby merged 1 commit intohalo-dev:masterfrom
JohnNiang:bug/version-comparison

Conversation

@JohnNiang
Copy link
Copy Markdown
Member

What this PR does

Remove compare method and add a compatible method instead in Version class.

Which issue this PR fix

Fix #1704

/cc @halo-dev/sig-halo

@JohnNiang JohnNiang added the kind/bug Categorizes issue or PR as related to a bug. label Mar 5, 2022
@JohnNiang JohnNiang added this to the 1.5.x milestone Mar 5, 2022
Copy link
Copy Markdown
Member

@ruibaby ruibaby left a comment

Choose a reason for hiding this comment

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

LGTM

@ruibaby
Copy link
Copy Markdown
Member

ruibaby commented Mar 5, 2022

我们似乎还是有必要做主题的 max 和 min 版本验证的,光 require 一个最低版本还是不太完整。

@ruibaby ruibaby merged commit 03006e8 into halo-dev:master Mar 5, 2022
@JohnNiang JohnNiang deleted the bug/version-comparison branch March 5, 2022 12:24
winar-jin pushed a commit to winar-jin/halo that referenced this pull request Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Categorizes issue or PR as related to a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

alpha版本无法通过主题安装版本验证

3 participants