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

fgetcsvのテストを追加 #223

Merged
merged 5 commits into from
Apr 23, 2019
Merged

fgetcsvのテストを追加 #223

merged 5 commits into from
Apr 23, 2019

Conversation

nobuhiko
Copy link
Contributor

@nobuhiko nobuhiko commented Aug 16, 2018

winで落ちるテストを書いたつもりが落ちませんでした。
こういうことじゃないんだろうか?
#222

@coveralls
Copy link

coveralls commented Aug 16, 2018

Coverage Status

Coverage remained the same at 22.644% when pulling 40d42a3 on nobuhiko:add-test#222 into 6136c21 on EC-CUBE:improve/php7.

@nanasess
Copy link
Contributor

AppVeyor は、おそらく英語版 Windows なので再現しない可能性が
EC-CUBE/ec-cube#1780

@nobuhiko
Copy link
Contributor Author

@nanasess となると日本語版 win 環境持ってないのでお手上げでございます😅

@nanasess
Copy link
Contributor

@nobuhiko こちらでも環境作って見てみますね

@okazy okazy added this to the 2.17.0 milestone Aug 23, 2018
@nanasess
Copy link
Contributor

以下の設定をしてもシステムロケールは English_United States.1252 が設定される
nanasess@47073b0

AppVeyor では検証できないかも

@nobuhiko
Copy link
Contributor Author

検証出来ないので @nanasess さんに修正はおまかせします😂

@chihiro-adachi chihiro-adachi merged commit fb29970 into EC-CUBE:improve/php7 Apr 23, 2019
@chihiro-adachi
Copy link
Contributor

ありがとうございます、マージしました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants