Skip to content
Snippets Groups Projects

Linting: markdownlint-cli2

Merged Robin Candau requested to merge antiz/pacman-contrib:mdl into master
4 unresolved threads

Continuing the effort started in !32 (merged) and !33 (merged) by adding a pipeline job for mdl (task listed in #11).

This MR is based on the decisions that have been made in !32 (merged) (rules, separate job for mdl, using alpine as image, etc...).
It'll also have to be rebased against !32 (merged) and !33 (merged) once they're merged.

mdl isn't packaged for Alpine (yet?), so it is installed via rubygem within the pipeline for now.

Here's a run of the mdl job.
I'd happily help to fix/treat the "issues" raised by mdl once this MR is merged! :smile_cat:

I remain available if any changes are needed :slight_smile:

Edited by Robin Candau

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
7 15 - make
8 16 script: make check
17
18 .lint-workflow:
9 19 rules:
10 - if: '$CI_PIPELINE_SOURCE == "merge_request_event"'
20 - if: '$CI_PIPELINE_SOURCE == "merge_request_event" || ($CI_COMMIT_BRANCH && $CI_PIPELINE_SOURCE == "push")'
21 changes:
22 paths:
23 - $FILE_GLOB
24 compare_to: 'master'
11 25 - if: '$CI_COMMIT_BRANCH && $CI_OPEN_MERGE_REQUESTS && $CI_PIPELINE_SOURCE == "push"'
12 26 when: never
13 - if: '$CI_COMMIT_BRANCH && $CI_PIPELINE_SOURCE == "push"'
27
28 mdl:
  • 18 .lint-workflow:
    9 19 rules:
    10 - if: '$CI_PIPELINE_SOURCE == "merge_request_event"'
    20 - if: '$CI_PIPELINE_SOURCE == "merge_request_event" || ($CI_COMMIT_BRANCH && $CI_PIPELINE_SOURCE == "push")'
    21 changes:
    22 paths:
    23 - $FILE_GLOB
    24 compare_to: 'master'
    11 25 - if: '$CI_COMMIT_BRANCH && $CI_OPEN_MERGE_REQUESTS && $CI_PIPELINE_SOURCE == "push"'
    12 26 when: never
    13 - if: '$CI_COMMIT_BRANCH && $CI_PIPELINE_SOURCE == "push"'
    27
    28 mdl:
    29 image: alpine
    30 before_script:
    31 - apk add ruby && gem install mdl
  • Robin Candau mentioned in merge request !33 (merged)

    mentioned in merge request !33 (merged)

  • Robin Candau marked this merge request as draft

    marked this merge request as draft

  • Robin Candau added 8 commits

    added 8 commits

    Compare with previous version

  • Daniel M. Capella added 19 commits

    added 19 commits

    Compare with previous version

  • Daniel M. Capella changed title from Draft: Linting: Mdl to Draft: Linting: markdownlint-cli2

    changed title from Draft: Linting: Mdl to Draft: Linting: markdownlint-cli2

  • added 1 commit

    Compare with previous version

  • Daniel M. Capella added 2 commits

    added 2 commits

    Compare with previous version

  • Daniel M. Capella marked this merge request as ready

    marked this merge request as ready

  • mentioned in commit 1535313e

  • Please register or sign in to reply
    Loading