Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

update docs, node, add docker & bash linting #173

Merged
merged 4 commits into from Sep 12, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
13 changes: 13 additions & 0 deletions .github/pull_request_template.md
@@ -0,0 +1,13 @@
# Summary of changes

Do any of the followings changes break current behaviour or configuration?

- **YES** / NO

## How changes have been tested

-

## List any unknowns

-
53 changes: 53 additions & 0 deletions .github/workflows/lint.yml
@@ -0,0 +1,53 @@
name: Lint

on:
pull_request:
types:
- opened
- edited
- reopened
- synchronize
workflow_dispatch:

permissions:
pull-requests: write

jobs:
lint-bash:
name: Lint Bash scripts
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: reviewdog/action-shellcheck@v1
with:
github_token: ${{ secrets.github_token }}
reporter: github-pr-review
level: warning
path: .
pattern: '*.sh'
fail_on_error: true

lint-dockerfile:
name: Lint Dockerfiles
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- name: hadolint
uses: reviewdog/action-hadolint@v1
with:
reporter: github-pr-review
level: warning
fail_on_error: true
hadolint_ignore: DL3016 DL3018 # Ignore pinning apk and npm packages to specific version with @

lint-actions:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- name: Lint Github Actions
uses: reviewdog/action-actionlint@v1
env:
REVIEWDOG_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
reporter: github-pr-review
2 changes: 1 addition & 1 deletion .github/workflows/main.yml
Expand Up @@ -7,7 +7,7 @@ jobs:
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@master
- uses: actions/checkout@v3
with:
fetch-depth: '0'
- name: Bump version and push tag
Expand Down
4 changes: 2 additions & 2 deletions Dockerfile
@@ -1,9 +1,9 @@
FROM node:12-alpine3.15
FROM node:16-alpine
LABEL "repository"="https://github.com/anothrNick/github-tag-action"
sbe-arg marked this conversation as resolved.
Show resolved Hide resolved
LABEL "homepage"="https://github.com/anothrNick/github-tag-action"
LABEL "maintainer"="Nick Sjostrom"

RUN apk update && apk add bash git curl jq && npm install -g semver
RUN apk --no-cache add bash git curl jq && npm install -g semver

COPY entrypoint.sh /entrypoint.sh

Expand Down
6 changes: 3 additions & 3 deletions README.md
Expand Up @@ -22,17 +22,17 @@ jobs:
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3
with:
fetch-depth: '0'
- name: Bump version and push tag
uses: anothrNick/github-tag-action@1.36.0
uses: anothrNick/github-tag-action@1.40.0
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
WITH_V: true
```

_NOTE: set the fetch-depth for `actions/checkout@v2` to be sure you retrieve all commits to look for the semver commit message._
_NOTE: set the fetch-depth for `actions/checkout@v2` or newer to be sure you retrieve all commits to look for the semver commit message._

#### Options

Expand Down