Skip to content

Commit

Permalink
Merge pull request fluidattacks#1384 from rohaquinlop/issue-1374
Browse files Browse the repository at this point in the history
feat(build): fluidattacks#1374 deprecate lint-markdown
  • Loading branch information
dsalaza4 authored Oct 22, 2024
2 parents 1cde20d + a0da1f9 commit 382fe21
Show file tree
Hide file tree
Showing 13 changed files with 0 additions and 187 deletions.
16 changes: 0 additions & 16 deletions .github/workflows/dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -264,22 +264,6 @@ jobs:
- name: /lintGitMailMap
run: nix-env -if . && m . /lintGitMailMap

linux_lintMarkdown_all:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@f095bcc56b7c2baf48f3ac70d6d6782f4f553222
- uses: docker://docker.io/nixos/nix@sha256:c3db4c484f6b1ee6c9bb8ca90307cfbeca8ef88156840911356a677eeaff4845
name: /lintMarkdown/all
with:
args: sh -c "chown -R root:root /github/workspace && nix-env -if . && m . /lintMarkdown/all"
macos_lintMarkdown_all:
runs-on: macos-latest
steps:
- uses: actions/checkout@f095bcc56b7c2baf48f3ac70d6d6782f4f553222
- uses: cachix/install-nix-action@6ed004b9ccb68dbc28e7c85bee15fa93dbd214ac
- name: /lintMarkdown/all
run: nix-env -if . && m . /lintMarkdown/all

linux_lintNix:
runs-on: ubuntu-latest
steps:
Expand Down
20 changes: 0 additions & 20 deletions .github/workflows/prod.yml
Original file line number Diff line number Diff line change
Expand Up @@ -402,26 +402,6 @@ jobs:
env:
CACHIX_AUTH_TOKEN: ${{ secrets.CACHIX_AUTH_TOKEN }}

linux_lintMarkdown_all:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@f095bcc56b7c2baf48f3ac70d6d6782f4f553222
- uses: docker://docker.io/nixos/nix@sha256:c3db4c484f6b1ee6c9bb8ca90307cfbeca8ef88156840911356a677eeaff4845
name: /lintMarkdown/all
with:
args: sh -c "chown -R root:root /github/workspace && nix-env -if . && m . /lintMarkdown/all"
env:
CACHIX_AUTH_TOKEN: ${{ secrets.CACHIX_AUTH_TOKEN }}
macos_lintMarkdown_all:
runs-on: macos-latest
steps:
- uses: actions/checkout@f095bcc56b7c2baf48f3ac70d6d6782f4f553222
- uses: cachix/install-nix-action@6ed004b9ccb68dbc28e7c85bee15fa93dbd214ac
- name: /lintMarkdown/all
run: nix-env -if . && m . /lintMarkdown/all
env:
CACHIX_AUTH_TOKEN: ${{ secrets.CACHIX_AUTH_TOKEN }}

linux_lintNix:
runs-on: ubuntu-latest
steps:
Expand Down
46 changes: 0 additions & 46 deletions docs/src/api/builtins/lint.md
Original file line number Diff line number Diff line change
Expand Up @@ -154,52 +154,6 @@ Example:
m . /lintGitMailMap
```

## lintMarkdown

Lints Markdown code with [Markdown lint tool](https://github.com/markdownlint/markdownlint).

Types:

- lintMarkdown (`attrsOf moduleType`): Optional.
Definitions of config and associated paths to lint.
Defaults to `{ }`.
- moduleType (`submodule`):
- config (`str`): Optional.
Path to the config file.
Defaults to [config.rb](https://github.com/fluidattacks/makes/blob/main/src/evaluator/modules/lint-markdown/config.rb).
- targets (`listOf str`): Required.
paths to lint with `config`.
- rulesets (`str`): Optional.
Path to the custom rulesets file.
Defaults to [rulesets.rb](https://github.com/fluidattacks/makes/blob/main/src/evaluator/modules/lint-markdown/rulesets.rb).

Example:

=== "makes.nix"

```nix
{
lintMarkdown = {
all = {
# You can pass custom configs like this:
# config = "/src/config/markdown.rb";
# You can pass custom rules like this:
# rulesets = "/src/config/rulesets.rb";
targets = [ "/" ];
};
others = {
targets = [ "/others" ];
};
};
}
```

=== "Invocation"

```bash
m . /lintMarkdown/all
```

## lintNix

Lints Nix code with [statix](https://github.com/nerdypepper/statix).
Expand Down
1 change: 0 additions & 1 deletion docs/src/contributing.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ If you're adding new files, make sure to `git add` them first.
- All changes must be documented in the same PR.
- You can run `m . /docs/dev` to serve the [docs site](https://makes.fluidattacks.tech)
on localhost.
- Make sure you lint the documentation with `m . /lintMarkdown/all`.

#### Adding tests

Expand Down
6 changes: 0 additions & 6 deletions makes.nix
Original file line number Diff line number Diff line change
Expand Up @@ -132,12 +132,6 @@
config = "/test/lint-commit-msg/lint-git-commit-msg-config.js";
};
lintGitMailMap = { enable = true; };
lintMarkdown = {
all = {
config = "/test/lint-markdown/config.rb";
targets = [ "/" ];
};
};
lintNix = {
enable = true;
targets = [ "/" ];
Expand Down
1 change: 0 additions & 1 deletion src/args/agnostic.nix
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,6 @@ let
lintClojure = import ./lint-clojure/default.nix self;
lintGitCommitMsg = import ./lint-git-commit-msg/default.nix self;
lintGitMailMap = import ./lint-git-mailmap/default.nix self;
lintMarkdown = import ./lint-markdown/default.nix self;
lintNix = import ./lint-nix/default.nix self;
lintPython = import ./lint-python/default.nix self;
lintPythonImports = import ./lint-python-imports/default.nix self;
Expand Down
15 changes: 0 additions & 15 deletions src/args/lint-markdown/builder.sh

This file was deleted.

12 changes: 0 additions & 12 deletions src/args/lint-markdown/default.nix

This file was deleted.

1 change: 0 additions & 1 deletion src/evaluator/modules/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@
(import ./lint-clojure/default.nix args)
(import ./lint-git-commit-msg/default.nix args)
(import ./lint-git-mailmap/default.nix args)
(import ./lint-markdown/default.nix args)
(import ./lint-nix/default.nix args)
(import ./lint-python/default.nix args)
(import ./lint-terraform/default.nix args)
Expand Down
3 changes: 0 additions & 3 deletions src/evaluator/modules/lint-markdown/config.rb

This file was deleted.

35 changes: 0 additions & 35 deletions src/evaluator/modules/lint-markdown/default.nix

This file was deleted.

1 change: 0 additions & 1 deletion src/evaluator/modules/lint-markdown/rulesets.rb

This file was deleted.

30 changes: 0 additions & 30 deletions test/lint-markdown/config.rb

This file was deleted.

0 comments on commit 382fe21

Please sign in to comment.