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

Win環境のXAMPPインストール記事を4.0.4に対応 #117

Merged
merged 3 commits into from
Jul 15, 2020

Conversation

tsumu-tsumu
Copy link
Contributor

XAMPPのインストール記事を4.0.4に対応しました。
phpのバージョンが7.4対応になることをメインに修正しています。

@okazy
Copy link
Contributor

okazy commented Jul 2, 2020

ありがとうございます!変更内容も問題ないと思います。
4.0.3の手順も残しておいていただいていますが、最新版の手順だけでも良いかと思いました。

@okazy okazy added the installation インストールや環境にまつわるもの label Jul 2, 2020
@okazy okazy added this to the 4.0 milestone Jul 2, 2020
@tsumu-tsumu
Copy link
Contributor Author

tsumu-tsumu commented Jul 3, 2020

SEO的には、残しておいたほうが良いと思って残していました。
消したほうが良い理由を教えていただけないでしょうか?
SEOとその理由を比べて良い選択を考えたいです。

ただ、今後はバージョンがアップするごとに過去記事が増えていくようにはしたくなくて、
別の仕組みを考えています。また相談にのっていただけますと助かります。

@okazy
Copy link
Contributor

okazy commented Jul 10, 2020

@tsumu-tsumu

SEO的には、残しておいたほうが良いと思って残していました。

SEOの観点は考えられてませんでした。

消しても良いと思ったのは、

  • メンテナンス対象のページが増える(間違いや変更があった場合などに編集対象のファイルが多くなる)
  • ドキュメントを見た人が迷ってしまう可能性がある

という点です。

@tsumu-tsumu
Copy link
Contributor Author

対応遅くなりすみません。
そうしましたら、4.0.3の記事とリンクは削除しました。
修正の名前が4.0.4になっていますが、4.0.3の間違いです汗
直しかたがわからず…。紛らわしくてすみません。
よろしくお願いいたします。

Copy link
Contributor

@okazy okazy left a comment

Choose a reason for hiding this comment

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

修正ありがとうございます。問題ないかと思います。

@okazy
Copy link
Contributor

okazy commented Jul 15, 2020

@tsumu-tsumu

--amend オプションをつけると直前のコミットを上書きできます。

git commit --amend

Sourcetree でも「最新のコミットを修正」を選択するとこで上書きコミットができます。

https://qiita.com/nnishimura/items/b5410e35f3a292a55244#amend%E3%82%B3%E3%83%9F%E3%83%83%E3%83%88%E3%83%A1%E3%83%83%E3%82%BB%E3%83%BC%E3%82%B8%E3%82%92%E6%9B%B8%E3%81%8D%E7%9B%B4%E3%81%97%E3%81%9F%E3%81%84%E6%99%82

一度 GitHub に push していた場合は --force を指定して push し直す必要があります。

git push --force origin piyopiyo

Sourcetree でもできるようです。

https://qiita.com/block/items/380015fc66296300f9e9#force-push

@tsumu-tsumu tsumu-tsumu force-pushed the 20200702_gui_win_404 branch from cd4e46e to c44ecf1 Compare July 15, 2020 07:41
@tsumu-tsumu
Copy link
Contributor Author

できました!
ありがとうございます!!

@okazy okazy merged commit efae863 into EC-CUBE:master Jul 15, 2020
@okazy
Copy link
Contributor

okazy commented Jul 15, 2020

ありがとうございます!取り込みました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement 改善 installation インストールや環境にまつわるもの
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants