wearerequired / lint-action Goto Github PK

When trying to run ESLint I always end up with this error

Verifying setup for ESLint…
Verified ESLint setup
Will use ESLint to check the files with extensions js
Linting files in /home/runner/work/aftershokz/aftershokz/scripts/ with ESLint…
Error: Error parsing ESLint JSON output: Unexpected end of JSON input.

Running ESLint locally with the cli works so I'm not sure what's causing this error.

Here's my yml file

name: Lint

on: pull_request

jobs:
  run-linters:
    name: Run linters
    runs-on: ubuntu-latest

    steps:
      - name: Check out Git repository
        uses: actions/checkout@v2

      - name: Set up Node.js
        uses: actions/setup-node@v1
        with:
          node-version: 12

      - name: Install Node.js dependencies
        run: npm install

      - name: Run linters
        uses: samuelmeuli/lint-action@v1
        with:
          github_token: ${{ secrets.github_token }}
          eslint: true
          eslint_dir: scripts/
          stylelint: true
          stylelint_dir: styles/

FIrst Thanks for the action, it works great for stylelint but somehow i canoot get it to work with ESLINT.

The task always fails (also tried to run only this task for ESLINT)

This is the error i get in the console of github actions.

Thanks for looking into this

data:{"message":"Invalid request.\n\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.\n\"end_line\", \"start_line\" weren't supplied.","documentation_url":"https://developer.github.com/v3/checks/runs/#create-a-check-run"}`
}
Error: Error trying to create GitHub check for ESLint: Received status code 422
at Object.createCheck (/home/runner/work/_actions/samuelmeuli/lint-action/v1.2.0/src/github.js:113:9)
at processTicksAndRejections (internal/process/task_queues.js:93:5)
/home/runner/work/_actions/samuelmeuli/lint-action/v1.2.0/src/index.js:15
throw new Error(Exiting because of unhandled promise rejection);
^

Error: Exiting because of unhandled promise rejection
at process. (/home/runner/work/_actions/samuelmeuli/lint-action/v1.2.0/src/index.js:15:8)
at process.emit (events.js:210:5)
at processPromiseRejections (internal/process/promises.js:201:33)
at processTicksAndRejections (internal/process/task_queues.js:94:32)
`

Currently, all files in a repository are being linted. The action should detect which files are part of the relevant commit/PR and only check these files.

shfmt is a great linter for shell scripts to ensure a coding style.

In this code pushChanges is triggered even if there are no new commits. I've noticed a couple of issues with this:

• it re-creates deleted branches
• it can cause an error if the branch has been updated since the build started, even if there is no autofix

Would be nice to trigger a git push only if there are new commits. And ideally the git push would quietly fail if the branch has been updated since presumably a new run will be triggered.

Sample log of ESLint bombing on push without any changes:

Creating annotations and auto-fixes works as expected when the code is on a branch in the same repository.

Unfortunately, it currently doesn't seem work with pull requests from forks: The action has no permission to push auto-fix changes or create annotations.

This might be less surprising for some users.

When creating a github workflow, it is sometimes useful to clone the repository (or multiple repositories) a level or two down from the default. e.g. to build and install a linter.

It would be useful to have a `repo-dir' option (defaulting to '.') from within which lint-action would run.

Hi @samuelmeuli, thanks for this GitHub Action. I really like the concept and the separation of concerns.

I‘m willing to provide a PR for adding a linter for PHP files via PHPCS. Would you be okay with that?

I tried out the rubocop linter in my project and I noticed that when I have warnings:

I still get a green check:

That's not the way that the typical rubocop setup works, usually those errors would return a non-zero exit code and fail the build. I wouldn't even know I needed to look for errors unless I click into the Detail screen.

I noticed that here you're only considering errors in the Check status:
https://github.com/samuelmeuli/lint-action/blob/a838ab16415bb1752bb9d77b4ce13fa80bfcaef6/src/github.js#L75

Would you consider failing on warnings as well? Or make that configurable?

This option could allow users to pass additional command-line arguments to the linting command. E.g. useful if users want GitHub checks to fail if there are warnings (see #10).

I just got this error on an action and I'm not sure what's causing it, running eslint --no-color --format json "." outputs no error.

 Error: ESLint error: Parsing error: Unexpected token ...
    at Function.parseOutput (/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/linters/eslint.js:86:12)
    at runAction (/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/index.js:86:30)
/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/index.js:16
	throw new Error(`Exiting because of unhandled promise rejection`);
	^

Error: Exiting because of unhandled promise rejection
    at process.<anonymous> (/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/index.js:16:8)
    at process.emit (events.js:210:5)
    at processPromiseRejections (internal/process/promises.js:201:33)
    at processTicksAndRejections (internal/process/task_queues.js:94:32)

I'm interested in adding ember-template-lint as its only our linter that's not currently supported by lint-action. Any objections?

Most probably, I'm missing something, but ESLint and Prettier does not stop workflow and have green checkmark even with errors.

As a result code with lint errors get published.

This is my config:

      - name: Run linters
        uses: samuelmeuli/lint-action@v1
        with:
          github_token: ${{ secrets.github_token }}
          eslint: true
          eslint_extensions: js,jsx,ts,tsx,vue
          eslint_args: --max-warnings 0
          prettier: true
          prettier_extensions: json,less,css,md,gql,graphql,html,yaml
          prettier_args: --ignore-path .eslintignore

By the way, this workflow is for push event.

Hello,

thanks for this action

I wanted to run flake8 and black but it seems that for some reason he try to lint/format all my deps in a folder called "/lib/python....." not sure why....

to avoid that i looked at a way to select the list of directories and files to run on but seems the _dir does not do the job with multiple input

is there a better way?

Hi,

Related to #31 and #50. I understand not failing main job is by design of this action. However, I could not figure out how to prevent next release step if there are lint errors.

Could you provide a basic example how to prevent next step if there are lint errors?

Thanks,

I just tried to use this action for SwiftLint, but I'm getting this error:

null
/home/runner/work/_actions/samuelmeuli/lint-action/v1.5.0/src/index.js:13
	throw new Error(`Exiting because of unhandled promise rejection`);
	^

Error: Exiting because of unhandled promise rejection
    at process.<anonymous> (/home/runner/work/_actions/samuelmeuli/lint-action/v1.5.0/src/index.js:13:8)
    at process.emit (events.js:210:5)
    at processPromiseRejections (internal/process/promises.js:201:33)
    at processTicksAndRejections (internal/process/task_queues.js:94:32)

Here's my job configuration:

  swiftlint:
    runs-on: ubuntu-latest
    steps:
    - name: Lint Action
      uses: samuelmeuli/[email protected]
      with:
        github_token: ${{ secrets.GITHUB_TOKEN }}
        swiftlint: true

Is there a flag to disable annotations in pull requests?

I was going through the codebase to see how feasible to add this feature because we currently need it.

Suggestion

I was thinking of adding an extra param [linter]_only_changed_files: boolean (we can decide on the best name to go with).

Then checking doing a quick diff of the current branch with

FILES = git diff --diff-filter=$filter --name-only $baseBranch | grep -E $extensions

where:

• $filter: can be defaulted to ACM (added, copied and modified), more: https://git-scm.com/docs/git-diff#Documentation/git-diff.txt---diff-filterACDMRTUXB82308203
• $baseBranch: we can get the base branch from event.pull_request.base.ref
• $extensions: e.g .js|.ts or we use the default extension from the lint configuration

I see you already swap branch for hasFork here: https://github.com/samuelmeuli/lint-action/blob/a820202c34156c11c68a7c534178ac66e12758b9/src/git.js#L22

We just need to do the same thing with the base repository:

• check if it's a fork, if so fetch it and add it to maybe base-fork remote locally
• else if it's not a fork, we don't need to fetch
• with the above steps, we can easily infer the base remote and branch name

Don't know if this is what you want to support @samuelmeuli else I can spin up a PR for this.

@samuelmeuli wdyt?

After upgrading from 0.7.9 to 1.1.0 our team noticed that some lint violations had crept in. They were all autofixable so I'm wondering if the diff detection is not working?

0.7.9:

Verifying setup for RuboCop…
Verified RuboCop setup
Will use RuboCop to check the files with extensions rb
Linting and auto-fixing files in /home/runner/work/project/project with RuboCop…
RuboCop found no issues (success)
Committing changes
Pushing changes to ***github.com/company/project.git
To https://github.com/company/project.git

It's not obvious from the log in 0.7.9 but it did push an autofix commit.

1.1.0:

Verifying setup for RuboCop…
Verified RuboCop setup
Will use RuboCop to check the files with extensions rb
Linting and auto-fixing files in /home/runner/work/project/project with RuboCop…
RuboCop found no issues (success)

In 1.1.0 no autofix commit was pushed

Probably something I messed up...

Both eslint and prettier says they found an error, but the step didn't fail and no annotation was added.

From jestjs/jest#9367

This project is so cool 👍

I would like to support Dart and with that also Flutter linting :)
Is this something that makes sense for this project?

Pylint is currently the most popular Python linter out there. Is there any chance that you will add support for it?

https://github.com/samuelmeuli/lint-action/blob/e92161e1607518185cb000036d625fb32b7bd4e9/src/linters/flake8.js#L72

Annotations are correctly created by are missing the first two characters of the path (in this situation hardware_ordering):

This causes the annotations to not appear on the pull request.
I am using file-changes-action to detect changed files and passing the results to flake8_args like flake8_args: ${{ steps.get_file_changes.outputs.files}}

https://github.com/git/git/blob/master/Documentation/RelNotes/2.23.0.txt

Git version 2.23.0 introduced the switch command.

When the change for forks was introduce there was a migration from checkout to switch in the code here. This causes builds to fail if the version of git provided is <2.23.0 (as is the case with some Ubuntu boxes still on any <20/Focal release).

Right now the linter Check is re-using the linter name for the output title which results in a view like this:

A more typical title would include a summary of the results:

I'd suggest something like "No errors found" or "1 error and 2 warnings" etc as the Check title. Might work to use the same value you're using for summary now.

Tools like ESLint and Prettier offer commands to automatically fix (some of) the detected linting/formatting issues. This action should allow users to run these commands on their commits/PRs.

I'm using following the README.md example for javascript-example-eslint-and-prettier, but the workflow shows unexpected input, checking the inputs that's correct that input is not expected. Also it's trying to push changes to .github folder but not sure why. It's a private repo

Here is the error:

! [remote rejected] master -> master (refusing to allow a GitHub App to create or update workflow `.github/workflows/Lint.yml` without `workflows` permission)

This is my lint.yml file:

name: Lint
on: [push, pull_request]
jobs:
  node-lint-tests:
    runs-on: ${{ matrix.os }}
    if: "!contains(github.event.head_commit.message, 'ci skip')"

    strategy:
      matrix:
        os: [ 'ubuntu-latest']
        node: [10, 12]

    steps:
      - name: Check out Git repository
        uses: actions/checkout@v2

      - name: Set up Node.js
        uses: actions/setup-node@v1
        with:
          node-version: ${{ matrix.node }}

      - name: install node_modules
        run: yarn install --frozen-lockfile

      - name: Run linters
        uses: samuelmeuli/lint-action@v1
        with:
          github_token: ${{ secrets.github_token }}
          eslint: true
          prettier: true
          auto_fix: true

Hello.

First of all, thanks for the project!

I have a monorepo with two folders: app and server. I have a workflow for the app and another one for the server because the config is different.

When running the workflows, I noticed the following:

• The app one has an output with errors but is successful (neither ESLint nor Prettier cjecks are created).
• The server one doesn't have errors but fails (both ESLint and Prettier checks are created).
• All errors listed in the ESLint check (created when running the server one) belong to the app folder.

Because running only one of the workflows behaves as expected, I know that the workflows are properly set.

The server workflow is running first, so I think the issue is that, when the app workflow runs, ESLint and Prettier checks get overwritten.

Is there any workaround?

I think a way of solving it could be accepting an optional input to set the name for the created checks. Let me know if this sounds good and I can open a PR.

Thanks!

Error: ESLint error: Parsing error: The keyword 'const' is reserved
at Function.parseOutput (/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/linters/eslint.js:86:12)
at runAction (/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/index.js:86:30)

Hi,

I followed up the configuration for Python Setup but it breaks when it starts linting process.
PFB Error trace

Adding auth information to Git remote URL
/bin/sh: 1: Syntax error: "(" unexpected
Error: Command failed: git remote set-url origin https://jayanthchandra:***@github.com/jayanthchandra/boilerPlate-flask.git
/bin/sh: 1: Syntax error: "(" unexpected

at checkExecSyncError (child_process.js:621:11)
at execSync (child_process.js:657:15)
at run (/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/utils/action.js:69:18)
at Object.checkOutRemoteBranch (/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/git.js:17:3)
at runAction (/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/index.js:54:7)
at Object.<anonymous> (/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/index.js:113:1)
at Module._compile (internal/modules/cjs/loader.js:959:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:995:10)
at Module.load (internal/modules/cjs/loader.js:815:32)
at Function.Module._load (internal/modules/cjs/loader.js:727:14) {

status: 2,
signal: null,
output: [ null, '', '/bin/sh: 1: Syntax error: "(" unexpected\n' ],
pid: 2555,
stdout: '',
stderr: '/bin/sh: 1: Syntax error: "(" unexpected\n'
}
/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/index.js:13
throw new Error(Exiting because of unhandled promise rejection);

Error: Exiting because of unhandled promise rejection
at process. (/home/runner/work/_actions/samuelmeuli/lint-action/v1/src/index.js:13:8)
at process.emit (events.js:210:5)
at processPromiseRejections (internal/process/promises.js:201:33)
at processTicksAndRejections (internal/process/task_queues.js:94:32)

Is there any workaround for this ?

I tried both v1 and v1.5 but still has the same error.

Hi,

I'm sorry for posting the question here.
I'm getting to know github actions. I wonder what is the advantage of this action over simple run: yarn lint ?

Hi there,

I'm trialling this action with the default ESLint setup that comes with GatsbyJS, and i'm receiving the below error. Any idea's or is this a bug?

Here is a link to their eslint file

Thanks!

Thoughts on including RuboCop? https://rubocop.readthedocs.io

We've got a large legacy repo and I'd like to implement a Github Action to only lint changed files. Running a linter on the entire repo is not currently a possibility.

I'm using https://github.com/trilom/file-changes-action to detect these files, but it looks like the lint action has no way of passing them into the linters.

E.g. for Rubocop, it automatically adds a "." to the command which forces all files under the directory to be linted.

It'd be great if we could make this a general feature. Failing that I can look at fixing rubocop so it allows for passed-in files (the "." should be redundant).

Run samuelmeuli/[email protected]
Fetching all remote Git branches
From https://github.com/Netflix/dispatch
Checking out the "remove_unused_args" branch
 * [new branch]      bugfix/stable-time -> origin/bugfix/stable-time
 * [new branch]      develop            -> origin/develop
 * [new branch]      docs               -> origin/docs
 * [new branch]      feature/dashboard  -> origin/feature/dashboard
 * [new branch]      feature/update-incident-document -> origin/feature/update-incident-document
 * [new branch]      master             -> origin/master
error: pathspec 'remove_unused_args' did not match any file(s) known to git
Error: Command failed: git checkout remove_unused_args
error: pathspec 'remove_unused_args' did not match any file(s) known to git

    at checkExecSyncError (child_process.js:621:11)
    at execSync (child_process.js:657:15)
    at run (/home/runner/work/_actions/samuelmeuli/lint-action/v1.2.0/src/utils/action.js:69:18)
    at Object.checkOutBranch (/home/runner/work/_actions/samuelmeuli/lint-action/v1.2.0/src/git.js:9:2)
    at runAction (/home/runner/work/_actions/samuelmeuli/lint-action/v1.2.0/src/index.js:35:7)
    at Object.<anonymous> (/home/runner/work/_actions/samuelmeuli/lint-action/v1.2.0/src/index.js:93:1)
    at Module._compile (internal/modules/cjs/loader.js:959:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:995:10)
    at Module.load (internal/modules/cjs/loader.js:815:32)
    at Function.Module._load (internal/modules/cjs/loader.js:727:14) {
  status: 1,
  signal: null,
  output: [
    null,
    '',
    "error: pathspec 'remove_unused_args' did not match any file(s) known to git\n"
  ],
  pid: 3775,
  stdout: '',
  stderr: "error: pathspec 'remove_unused_args' did not match any file(s) known to git\n"
}
/home/runner/work/_actions/samuelmeuli/lint-action/v1.2.0/src/index.js:15
	throw new Error(`Exiting because of unhandled promise rejection`);
	^

Workflow file here: https://github.com/Netflix/dispatch/blob/develop/.github/workflows/list_and_test.yml

I'm attempting to run an auto-fixing Black linter, and it changes files but then fails to actually commit them.

name: Run Linters

on:
  pull_request:
    paths:
      - 'c***p/**.py'

jobs:
  run-python-linter:
    name: Run Python Linter
    runs-on: ubuntu-latest

    steps:
      - name: Check out Git repository
        uses: actions/checkout@v2

      - name: Set up Python
        uses: actions/setup-python@v1
        with:
          python-version: 3.7

      - name: Install Python dependencies
        run: pip install black

      - name: Run linters
        uses: samuelmeuli/lint-action@v1
        with:
          github_token: ${{ secrets.github_token }}

          black: true
          black_args: -t py37 -l 120 --exclude "/*scripts*"
          black_dir: c****d
          auto_fix: true
          commit_message: Fix code style issues with ${linter}

When I push to a branch that is in pull request, I get the following:

Will use Black to check the files with extensions py
Linting and auto-fixing files in /home/runner/work/Base/B***ad with Black…
reformatted /home/runner/work/Base/Base/c***views.py
All done! ✨ 🍰 ✨
1 file reformatted, 170 files left unchanged.
Black found no issues (success)

SHA of last commit is "704312c7d00bc047a4a552f56760a3f6f8e7dd2a"
Creating GitHub check with 0 annotations for Flake8…
Creating GitHub check with 0 annotations for Black…

I've starred out some file names and directories, sorry for the silly paranoia but it is a private corporate repository. Nonetheless, everything seems to be operating fine, it's finding a file that is indeed out of whack and is reformatting it - it just doesn't seem to recognize that there have been Git changes. I'm guessing this is because of the change to check for changes before committing? When I run Black and then "git diff-index --cached HEAD" on my own machine, it tells me there aren't changes even though there are. Is it possible that the "--cached" option is causing problems?

A potential confounding issue is that the pull request is the one that actually includes the linter workflow.

Following the example from the documentation, I created a new workflow file for linting, called lint.yml. I have another workflow file named ci.yml that handles running tests (and, previously, running yarn lint on the project). Maybe this is intended behavior, but when the "Run linters" job runs, it creates new jobs for each linter in the "CI" workflow, rather than the "Lint" workflow. Consequently, it's a bit confusing in the UI. The "Lint" workflow is missing jobs our team thinks it should have while "CI" has jobs that it shouldn't.

Hi, I'm trying to use a workflow with a lint job and a build job. The build job is set to need the lint to complete successfully to begin so that I don't waste time building if linting returned issues. When linting though, that step is marked successfully completed even if the linters found issues. Is this expected behaviour? Shouldn't the step be marked fail if the linter returns issues?

name: STAGING - Lint and Build

on:
  push:
    branches-ignore:
      - master
jobs:
  Lint:
    runs-on: ubuntu-latest

    steps:
      - name: Checkout
        uses: actions/checkout@v2

      - name: "Setup Node.js"
        uses: actions/setup-node@v1
        with:
          node-version: 12.x
      
      - name: "Install Node Packages"
        run: npm ci

      - name: "Lint - Sass & Javascript"
        uses: samuelmeuli/[email protected]
        with:
          github_token: ${{ secrets.github_token }}
          eslint: true
          stylelint: true

  Build:
    runs-on: ubuntu-latest
    needs: Lint # Only run the build if linting suceeded.

    steps:
      - name: Test
        run: ls -la

Let me know if I'm missing something in the setup? I ran yarn install and npm install before running yarn test.

The context is I'm adding a new linter for HTML, and was hoping to test its functionality.

Test Suites: 1 failed, 5 passed, 6 total
Tests:       52 failed, 20 passed, 72 total
Snapshots:   0 total
Time:        14.753 s

Full trace: https://pastebin.com/hexGYsWg
Repo: https://github.com/terencedignon/lint-action

It would be helpful for projects which have CLA requirements to be able to specify the author of the fix commit.

The linter for slim: https://github.com/sds/slim-lint

we run eslint --config .eslint-config.js but i get an error:

Error: Error parsing ESLint JSON output: Unexpected end of JSON input. Output: ""
    at Function.parseOutput (/home/runner/work/_actions/samuelmeuli/lint-action/v1.5.2/src/linters/eslint.js:72:10)
    at runAction (/home/runner/work/_actions/samuelmeuli/lint-action/v1.5.2/src/index.js:85:30)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
/home/runner/work/_actions/samuelmeuli/lint-action/v1.5.2/src/index.js:13
	throw new Error(`Exiting because of unhandled promise rejection`);
	^

Error: Exiting because of unhandled promise rejection
    at process.<anonymous> (/home/runner/work/_actions/samuelmeuli/lint-action/v1.5.2/src/index.js:13:8)
    at process.emit (events.js:210:5)
    at processPromiseRejections (internal/process/promises.js:201:33)
    at processTicksAndRejections (internal/process/task_queues.js:94:32)

It seems lint-action doesn't relay lint warnings generated by swift-format official.

Run log:

Verifying setup for swift-format…
Verified swift-format setup
Will use swift-format to check the files with extensions swift
Linting files in /__w/repo/repo with swift-format…
/__w/repo/repo/ThumbnailContainerViewController.swift:15:1: warning: [BeginDocumentationCommentWithOneLineSummary]: terminate this sentence with a period: "Adopted by inner/child view controllers to specify  positioning of the thumnail overlay"
/__w/repo/repo/ThumbnailContainerViewController.swift:47:39: warning: [NeverUseImplicitlyUnwrappedOptionals]: use ThumbnailContainerView or ThumbnailContainerView? instead of ThumbnailContainerView!
swift-format found no issues (success)

SHA of last commit is "b979..."
Creating GitHub check with 0 annotations for swift-format…
swift-format check created successfully

And the parameters to configure the action:

github_token: ***
    auto_fix: false
    swift_format_official: true
    git_name: Lint Action
    commit_message: Fix code style issues with ${linter}
    stylelint: false
    stylelint_extensions: css,sass,scss
    gofmt: false
    gofmt_extensions: go
    golint: false
    golint_extensions: go
    eslint: false
    eslint_extensions: js
    prettier: false
    prettier_extensions: css,html,js,json,jsx,md,sass,scss,ts,tsx,vue,yaml,yml
    xo: false
    xo_extensions: js
    black: false
    black_extensions: py
    flake8: false
    flake8_extensions: py
    mypy: false
    mypy_extensions: py
    rubocop: false
    rubocop_extensions: rb
    swiftformat: false
    swiftformat_extensions: swift
    swift_format_lockwood: false
    swift_format_lockwood_extensions: swift
    swift_format_official_extensions: swift
    swiftlint: false
    swiftlint_extensions: swift

See: https://github.com/autostyle/autostyle

See https://github.com/rust-lang/rust-clippy

See https://github.com/rust-lang/rustfmt

It would be nice to have support for swift-format (https://github.com/apple/swift-format), which should not be confused with swiftformat (https://github.com/nicklockwood/SwiftFormat). I think that a challenge here may be the naming.

Hi,

I've been using lint-action for a while now and most of the time it's been fine, however, I've noticed after creating a new "feature" and my linitng task is failing. I get the following output:

Im not entirely sure what I can do to solve this issue, is there a way to get a new github token for the current branch or is this related to the token scoping Github has put in place ?

I really want to add this to our Github flow at work, I've used in all my personal projects and I love it, but I keep getting errors like this and I don't understand them fully or how to fix them.

Any help would greatly be appreciated 👍

Project Link: https://github.com/AlexMachin1997/TheOpenMovieDB-React-Example/actions