[Lunr.Vortex] General: Update ApnsPHP to master branch

Review Request #757 — Created April 6, 2021 and submitted

m.vanberkum
Lunr.Vortex
fix/ApnsPHPVersion
2606d9f...
lunr

Update ApnsPHP to master branch

Unit tests.

  • 0
  • 0
  • 5
  • 0
  • 5
Description From Last Updated
There are no open issues
m.vanberkum
pprkut
  1. 
      
  2. The issue has been resolved. Show all issues

    Why not \ApnsPHP\Message\Exception?

  3. The issue has been resolved. Show all issues

    Why that rename? \ApnsPHP\Exception would have been fine

  4. The issue has been resolved. Show all issues

    Absolute Path please. Relative path only for classes in the same folder.

  5. 
      
m.vanberkum
m.vanberkum
pprkut
  1. 
      
  2. The issue has been resolved. Show all issues

    I think for consistency we should probably alias this as ApnsException ot ApnsPHPException or some such.

  3. The issue has been resolved. Show all issues

    ditto

  4. 
      
m.vanberkum
smillernl
  1. Ship It!
  2. 
      
pprkut
Review request changed

Status: Closed (submitted)

Change Summary:

Merged into master

Loading...