Support for log file datetime separators

SeasX / SeasLog

An effective,fast,stable log extension for PHP.http://pecl.php.net/package/SeasLog http://php.net/SeasLog

http://seasx.github.io/SeasLog/

Other

1.2k stars 276 forks source link

Closed her-cat closed 2 years ago

her-cat commented 3 years ago

Related issues: #296

Rock-Lee-520 commented 2 years ago

感谢你提的PR，我在做回归测试时，发现php7.2 回归测试通过，但是php8.0 回归测试时， tests/016.phpt、tests/017.phpt出现断言错误，请处理一下，或者我提交给你， fix该问题后再提交merge

her-cat commented 2 years ago

好的，我处理一下这个问题

Rock-Lee-520 commented 2 years ago

好的，我处理一下这个问题

这个问题有新的进展么？是否需要协助？

her-cat commented 2 years ago

上面提的那个问题已经处理了，您这边回归测试还有问题吗？

Rock-Lee-520 commented 2 years ago

analyzerCount 跟analyzerDetail 函数未通过单测哦

代码：

Rock-Lee-520 commented 2 years ago

上面提的那个问题已经处理了，您这边回归测试还有问题吗？

her-cat commented 2 years ago

能否发一下错误信息呢，我这边单测都是通过的。

her-cat commented 2 years ago

@Rock-liyi

wujunze commented 2 years ago

LGTM

Rock-Lee-520 commented 2 years ago

@Rock-liyi tests/016.phpt

tests/017.phpt

Rock-Lee-520 commented 2 years ago

@Rock-liyi

Rock-Lee-520 commented 2 years ago

能否发一下错误信息呢，我这边单测都是通过的。

tests/016.phpt

tests/017.phpt

Rock-Lee-520 commented 2 years ago

我知道什么原因了

配置上seaslog.default_datetime_format后，单测需要调整

Rock-Lee-520 commented 2 years ago

可以改一下这两个单测的实现