Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add solutions to lc/lcof2 problems: No. 008 #630

Merged
merged 4 commits into from
Dec 12, 2021
Merged

feat: add solutions to lc/lcof2 problems: No. 008 #630

merged 4 commits into from
Dec 12, 2021

Conversation

Yousazoe
Copy link
Contributor

No description provided.

@yanglbme
Copy link
Member

@Yousazoe 这个提交有点问题,把上一次的 pr 内容也再提交了一边。

image

我的建议是 fork 本项目,fork 后的项目的 main 分支随时与上游 doocs/leetcode main 分支保持同步。你每次可以基于本地的 main 分支最新代码 checkout -b 一个新的分支,然后提 pr。

@yanglbme
Copy link
Member

本次 pr 我先合并,下次可以规范起来哈,不然新提交的 pr 夹带着已经 merge 进主分支的代码,review 起来不是很方便直观。

@yanglbme yanglbme merged commit 3869b28 into doocs:main Dec 12, 2021
@Yousazoe
Copy link
Contributor Author

本次 pr 我先合并,下次可以规范起来哈,不然新提交的 pr 夹带着已经 merge 进主分支的代码,review 起来不是很方便直观。


请问这样是不是没有问题了,昨天pr的时候忘记同步了,抱歉

@yanglbme
Copy link
Member

可以提个 pr 看看哈

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