Skip to content

Commit 6ea674d

Browse files
authored
Merge pull request #49 from laravel-notification-channels/experimental-8.1
PHP 8.1 in tests workflow as experimental
2 parents ed68da1 + 717e850 commit 6ea674d

File tree

4 files changed

+11
-12
lines changed

4 files changed

+11
-12
lines changed

.github/workflows/tests.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,17 @@ on:
77
jobs:
88
tests:
99
runs-on: ubuntu-latest
10+
continue-on-error: ${{ matrix.experimental }}
1011
strategy:
1112
fail-fast: true
1213
matrix:
1314
php: [7.3, 7.4, 8.0]
1415
stability: [prefer-lowest, prefer-stable]
16+
experimental: [false]
17+
include:
18+
- php: 8.1
19+
stability: prefer-stable
20+
experimental: true
1521

1622
name: Tests on PHP ${{ matrix.php }} - ${{ matrix.stability }}
1723

src/Pushbullet.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ private function getHeaders(): array
5656
* Send request to Pushbullet API.
5757
*
5858
* @param array $params
59-
*
6059
* @return \Psr\Http\Message\ResponseInterface
6160
*/
6261
public function send($params): ResponseInterface

src/PushbulletChannel.php

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@ public function __construct(Pushbullet $pushbullet)
2525
/**
2626
* Send the given notification.
2727
*
28-
* @param mixed $notifiable
29-
* @param \Illuminate\Notifications\Notification $notification
28+
* @param mixed $notifiable
29+
* @param \Illuminate\Notifications\Notification $notification
3030
*
3131
* @throws \NotificationChannels\Pushbullet\Exceptions\CouldNotSendNotification
3232
*/
@@ -44,8 +44,7 @@ public function send($notifiable, Notification $notification): void
4444

4545
/**
4646
* @param $notifiable
47-
*
48-
* @return \NotificationChannels\Pushbullet\Targets\Targetable|void
47+
* @return \NotificationChannels\Pushbullet\Targets\Targetable|void
4948
*/
5049
private function getTarget($notifiable): ?Targetable
5150
{

src/PushbulletMessage.php

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,7 @@ class PushbulletMessage
4343
public $url;
4444

4545
/**
46-
* @param string $message
47-
*
46+
* @param string $message
4847
* @return static
4948
*/
5049
public static function create($message): self
@@ -53,7 +52,7 @@ public static function create($message): self
5352
}
5453

5554
/**
56-
* @param string $message
55+
* @param string $message
5756
*/
5857
public function __construct($message)
5958
{
@@ -62,7 +61,6 @@ public function __construct($message)
6261

6362
/**
6463
* @param \NotificationChannels\Pushbullet\Targets\Targetable $targetable
65-
*
6664
* @return $this
6765
*/
6866
public function target(Targetable $targetable): self
@@ -100,7 +98,6 @@ public function link(): self
10098
* Set notification title.
10199
*
102100
* @param string $title
103-
*
104101
* @return $this
105102
*/
106103
public function title($title): self
@@ -114,7 +111,6 @@ public function title($title): self
114111
* Set notification message.
115112
*
116113
* @param string $message
117-
*
118114
* @return $this
119115
*/
120116
public function message($message): self
@@ -128,7 +124,6 @@ public function message($message): self
128124
* Set notification url (if notification is of `link` type).
129125
*
130126
* @param string $url
131-
*
132127
* @return $this
133128
*/
134129
public function url($url): self

0 commit comments

Comments
 (0)