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

#3978 #4110

Merged
merged 8 commits into from
Jan 10, 2025
Merged

#3978 #4110

merged 8 commits into from
Jan 10, 2025

Conversation

IwasakiRyuichi
Copy link
Contributor

@ryuring

午前に言われた対応行いました。
BcThemeSampleにBcThemeTestHelperという2つ目のヘルパーを新しく作成し、コピーを実行するとBcThemeTestCopyHelperが作られ、正常に画面の表示も確認できましたので機能実装はできていると思われます。
お手隙の際にご確認いただければと思います。よろしくお願いいたします。

@ryuring
Copy link
Collaborator

ryuring commented Jan 9, 2025

@IwasakiRyuichi コンフリクトしているので修正をお願いします。
本家の最新版をプルしてからプッシュすると解消されると思います。

@IwasakiRyuichi
Copy link
Contributor Author

@ryuring
mainをpullして、再度pushしました。

@ryuring
Copy link
Collaborator

ryuring commented Jan 9, 2025

@IwasakiRyuichi コンフリクトのままですね。

mainをpullして、再度pushしました。

まず、このレポジトリは、main というブランチは存在せず、master になります。
ただ、masterではなく、その上で最新化しないといけないのは、5.1.x です。

まずは、本家を追加して、本家の 5.1.x をpull する必要があります。

# リモートを追加(basercms部分は任意の名前で可)
git remote add basercms [email protected]:baserproject/basercms.git
# 5.1.x を pull
git pull basercms 5.1.x

@IwasakiRyuichi
Copy link
Contributor Author

@ryuring
helperの複数コピー確認用に仮で作成していた、余分なhelperを削除しました。

@ryuring
Copy link
Collaborator

ryuring commented Jan 10, 2025

一旦マージしますので、ユニットテストの実装をお願いします。 @IwasakiRyuichi

@ryuring ryuring merged commit a8bd016 into baserproject:5.1.x Jan 10, 2025
1 check passed
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