Skip to content

feat: add r-hub workflow #1249

feat: add r-hub workflow

feat: add r-hub workflow #1249

Workflow file for this run

---
name: SuperLinter 🦸‍♀️
on:
push:
branches:
- main
pull_request:
types:
- opened
- synchronize
- reopened
- ready_for_review
branches:
- main
workflow_dispatch:
workflow_call:
inputs:
lintr_error_on_lint:
description: Produce an error when lints are found
required: false
type: boolean
default: true
lint-all-files:
description: Lint all files every time
default: false
required: false
type: boolean
concurrency:
group: lint-${{ github.event.pull_request.number || github.ref }}
cancel-in-progress: true
jobs:
lint:
name: SuperLinter 🦸‍♂️
runs-on: ubuntu-latest
if: >
!contains(github.event.commits[0].message, '[skip linter]')
&& github.event.pull_request.draft == false
steps:
- name: Get branch names 🌿
id: branch-name
uses: tj-actions/branch-names@v7
- name: Checkout repo (PR) 🛎
uses: actions/[email protected]
if: github.event_name == 'pull_request'
with:
ref: ${{ steps.branch-name.outputs.head_ref_branch }}
repository: ${{ github.event.pull_request.head.repo.full_name }}
fetch-depth: 0
- name: Checkout repo 🛎
uses: actions/[email protected]
if: github.event_name != 'pull_request'
with:
ref: ${{ steps.branch-name.outputs.head_ref_branch }}
fetch-depth: 0
- name: Check commit message 💬
run: |
git config --global --add safe.directory $(pwd)
export head_commit_message="$(git show -s --format=%B | tr '\r\n' ' ' | tr '\n' ' ')"
echo "head_commit_message = $head_commit_message"
if [[ $head_commit_message == *"$SKIP_INSTRUCTION"* ]]; then
echo "Skip instruction detected - cancelling the workflow."
exit 1
fi
shell: bash
env:
SKIP_INSTRUCTION: "[skip linter]"
- name: Override SuperLinter configs 💾
run: |
# Override markdownlint default config
[ ! -e .markdownlint.yaml ] && cat > .markdownlint.yaml <<EOF
###############
# Rules by id #
###############
MD004: false # Unordered list style
MD007:
indent: 2 # Unordered list indentation
MD013:
line_length: 800 # Line length
MD026:
punctuation: ".,;:!。,;:" # List of not allowed
MD029: false # Ordered list item prefix
MD033: false # Allow inline HTML
MD036: false # Emphasis used instead of a heading
#################
# Rules by tags #
#################
blank_lines: true # Error on blank lines
EOF
shell: bash
- name: Lint Code Base 🧶
uses: super-linter/super-linter/slim@v5
env:
LINTER_RULES_PATH: /
DEFAULT_BRANCH: ${{ github.event.repository.default_branch }}
FILTER_REGEX_EXCLUDE: NEWS.md|design/.*|testthat/_snaps/.*
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
VALIDATE_ALL_CODEBASE: false
VALIDATE_BASH: true
VALIDATE_DOCKERFILE: true
VALIDATE_MARKDOWN: true
MARKDOWN_CONFIG_FILE: .markdownlint.yaml
VALIDATE_YAML: true
lint-r-code:
name: Lint R code 🧶
runs-on: ubuntu-latest
if: >
!contains(github.event.commits[0].message, '[skip linter]')
&& github.event.pull_request.draft == false
container:
image: ghcr.io/insightsengineering/rstudio:latest
steps:
- name: Checkout repo (PR) 🛎
uses: actions/[email protected]
if: github.event_name == 'pull_request'
with:
ref: ${{ github.event.pull_request.head.sha }}
fetch-depth: 0
- name: Checkout repo 🛎
uses: actions/[email protected]
if: github.event_name != 'pull_request'
with:
fetch-depth: 0
- name: Check commit message 💬
run: |
git config --global --add safe.directory $(pwd)
export head_commit_message="$(git show -s --format=%B | tr '\r\n' ' ' | tr '\n' ' ')"
echo "head_commit_message = $head_commit_message"
if [[ $head_commit_message == *"$SKIP_INSTRUCTION"* ]]; then
echo "Skip instruction detected - cancelling the workflow."
exit 1
fi
shell: bash
env:
SKIP_INSTRUCTION: "[skip linter]"
- name: Changed files 🖋️
if: github.event_name != 'workflow_dispatch'
id: files
uses: Ana06/[email protected]
with:
format: 'json'
filter: '*'
- name: Lint 🧶
run: |
github_event_name <- "${{ github.event_name }}"
cat(paste0("GitHub event name = ", github_event_name, "\n"))
exclusions_list <- NULL
if (!(identical("${{ inputs.lint-all-files }}", "true") || identical(github_event_name, "workflow_dispatch"))) {
changed_files <- jsonlite::fromJSON('${{ steps.files.outputs.added_modified }}')
all_files <- list.files(recursive = TRUE)
exclusions_list <- as.list(setdiff(all_files, changed_files))
cat("Linting only changed files:\n")
cat(changed_files)
cat("\n\n")
} else {
cat("Linting all files.\n")
}
lints <- lintr::lint_package(exclusions = exclusions_list)
if (length(lints) > 0L) {
print(lints)
if (identical("${{ inputs.lintr_error_on_lint }}", "true")) {
stop("Lints detected. Please review and adjust code according to the comments provided.", call. = FALSE)
}
}
shell: Rscript {0}