Merge pull request #1 from comes/main

Merge catchup. Confusing day.
This commit is contained in:
Marlon van der Linde
2023-11-01 08:51:14 +02:00
committed by GitHub
2 changed files with 25 additions and 6 deletions

23
.github/workflows/pint.yml vendored Normal file
View File

@@ -0,0 +1,23 @@
name: PHP Linting (Pint)
on:
workflow_dispatch:
push:
branches-ignore:
- 'dependabot/npm_and_yarn/*'
jobs:
phplint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
with:
fetch-depth: 2
- name: "laravel-pint"
uses: aglipanci/laravel-pint-action@2.3.0
with:
preset: laravel
- name: Commit changes
uses: stefanzweifel/git-auto-commit-action@v4
with:
commit_message: PHP Linting (Pint)
skip_fetch: true

View File

@@ -16,6 +16,7 @@ class EcowittExportCommand extends Command
protected $endDate;
protected $ecowitt_account;
protected $ecowitt_passphrase;
/**
@@ -239,8 +240,6 @@ class EcowittExportCommand extends Command
/**
* fetch all available device IDs
* @param $session_id
* @return \Illuminate\Support\Collection
*/
protected function getDeviceIds($session_id): \Illuminate\Support\Collection
{
@@ -263,8 +262,6 @@ class EcowittExportCommand extends Command
/**
* Takes in a filename and an array associative data array and outputs a csv file
* @param string $fileName
* @param array $data
*/
protected function export(string $fileName, array $data)
{
@@ -279,8 +276,7 @@ class EcowittExportCommand extends Command
}
/**
* @param string $msg
* @param mixed ...$args
* @param mixed ...$args
*/
protected function debug(string $msg, ...$args)
{