Skip to content
New issue

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

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

Already on GitHub? # to your account

Add Message::time() method #114

Merged
merged 3 commits into from
Jul 1, 2024
Merged

Add Message::time() method #114

merged 3 commits into from
Jul 1, 2024

Conversation

vjik
Copy link
Member

@vjik vjik commented Jun 25, 2024

Q A
Is bugfix?
New feature? ✔️
Breaks BC?
Fixed issues comma-separated list of tickets # fixed by the PR, if any

Copy link

codecov bot commented Jun 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.55%. Comparing base (447337f) to head (6e2ef34).

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #114      +/-   ##
============================================
- Coverage     99.55%   99.55%   -0.01%     
+ Complexity      161      160       -1     
============================================
  Files            10       10              
  Lines           446      445       -1     
============================================
- Hits            444      443       -1     
  Misses            2        2              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vjik vjik requested a review from a team June 25, 2024 07:45
@vjik vjik added the status:code review The pull request needs review. label Jun 25, 2024
src/Message.php Outdated Show resolved Hide resolved
Co-authored-by: Alexander Makarov <sam@rmcreative.ru>
@vjik vjik merged commit ff5b29a into master Jul 1, 2024
20 of 21 checks passed
@vjik vjik deleted the message-time branch July 1, 2024 10:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
status:code review The pull request needs review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants