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

Avoid using docker_image pre-commit hook (portability) #218

Merged
merged 1 commit into from
Jun 17, 2024
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.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions .github/SECURITY.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,6 @@ The only supported version is the latest release.

## Reporting a Vulnerability

Send an email to [email protected] if you found any security
vulnerability that is better to not have a public bug raise right away.
Use [github reporting](https://github.com/pycontribs/mk/security) if you
found any security vulnerability that is better to not have a public bug raised
right away.
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,6 @@ report.html
site
src/mk.egg-info
src/mk/_version.py
.coverage
.coverage*
coverage.xml
CMakeFiles
10 changes: 3 additions & 7 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,4 @@
---
ci:
skip:
- markdownlint_docker
repos:
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.4.9
Expand All @@ -26,11 +23,10 @@ repos:
# entry: codespell --relative
args: [--relative, --no-progress, --no-summary]
name: Spell check with cspell

- repo: https://github.com/markdownlint/markdownlint.git
rev: v0.12.0
- repo: https://github.com/igorshubovych/markdownlint-cli
rev: v0.41.0
hooks:
- id: markdownlint_docker
- id: markdownlint
- repo: https://github.com/psf/black
rev: 24.4.2
hooks:
Expand Down
9 changes: 5 additions & 4 deletions docs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,10 @@ configuration file is needed as the tool will look for common tools used by the
repository and expose their commands.

`mk` is inspired by tools such [make][make], [waf][waf], [taskfile][taskfile],
[tox][tox], [npm][npm], [yarn][yarn] and [pre-commit][pre-commit], but **it does
not aim to replace them**. Instead, it aims to provide a unified interface for
calling them that is friendly even for those that never used these tools.
[tox][tox], [nox][nox], [npm][npm], [yarn][yarn] and [pre-commit][pre-commit],
but **it does not aim to replace them**. Instead, it aims to provide a unified
interface for calling them that is friendly even for those that never used
these tools.

## Installation

Expand Down Expand Up @@ -101,7 +102,7 @@ status or that the testing did not leave untracked files.
unknown. (#20)
- Configuration file where additional actions can be added. (#21)
- Dependencies between commands. While some tools support dependencies, many do
not. You should be able to declare that a specific command would run only
not. You should be able to declare that a specific command will run only
after another one already passed. (#22)
- Ability to generate CI/CD pipelines so the user would spend less time writing
non-portable configurations. (#23)
Expand Down
Loading