Skip to content

Commit 0cb94c4

Browse files
committed
feat: Use mdformat i/ of prettier
1 parent d841744 commit 0cb94c4

10 files changed

Lines changed: 571 additions & 620 deletions

File tree

.github/SECURITY.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
| Version | Supported |
66
| ------- | ------------------ |
77
| 2.x | :white_check_mark: |
8-
| <2.6.1 | :x: |
8+
| \<2.6.1 | :x: |
99

1010
## Reporting a Vulnerability
1111

.github/workflows/lib-checks.yml

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,20 @@ jobs:
105105
groups: "dev"
106106
- name: "Run bandit"
107107
run: bandit --recursive --exclude ./.venv --skip B101,B104,B105,B110,B307,B311,B404,B603,B607 .
108+
mdformat:
109+
needs: [ changes ]
110+
if: ${{ needs.changes.outputs.lib == 'true' && github.event_name != 'schedule' }}
111+
runs-on: ubuntu-latest
112+
steps:
113+
- name: "Checkout Repository"
114+
uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
115+
- name: "Setup uv"
116+
uses: ./.github/actions/setup-uv
117+
with:
118+
python-version: "3.14"
119+
groups: "dev"
120+
- name: "Run mdformat"
121+
run: git ls-files -z '*.md' | xargs -0 -r mdformat --check
108122
pylint:
109123
needs: [ changes ]
110124
if: ${{ needs.changes.outputs.lib == 'true' && github.event_name != 'schedule' }}

.mdformat.toml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
wrap = 100
2+
number = true

.pre-commit-config.yaml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -81,12 +81,13 @@ repos:
8181
# hooks:
8282
# - id: mypy
8383

84-
- repo: https://github.com/pre-commit/mirrors-prettier
85-
rev: v4.0.0-alpha.8
84+
- repo: https://github.com/hukkin/mdformat
85+
rev: 1.0.0
8686
hooks:
87-
- id: prettier
88-
args: [--prose-wrap=always, --print-width=88]
89-
exclude: \.(po|pot|yml|yaml)$
87+
- id: mdformat
88+
additional_dependencies:
89+
- mdformat-gfm==1.0.0
90+
- mdformat-gfm-alerts==2.1.0
9091
- repo: https://github.com/DanielNoord/pydocstringformatter
9192
rev: v1.0.0
9293
hooks:

.prettierrc

Lines changed: 0 additions & 3 deletions
This file was deleted.

CHANGELOG.md

Lines changed: 382 additions & 490 deletions
Large diffs are not rendered by default.

CONTRIBUTING.md

Lines changed: 58 additions & 63 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,9 @@
11
## Contributing to Pycord
22

3-
First off, thanks for taking the time to contribute. It makes the library substantially
4-
better. :+1:
3+
First off, thanks for taking the time to contribute. It makes the library substantially better. :+1:
54

6-
The following is a set of guidelines for contributing to the repository. These are
7-
guidelines, not hard rules.
5+
The following is a set of guidelines for contributing to the repository. These are guidelines, not
6+
hard rules.
87

98
## This is too much to read! I want to ask a question!
109

@@ -14,106 +13,102 @@ Generally speaking questions are better suited in our resources below.
1413
- [The FAQ in the documentation](https://docs.pycord.dev/en/master/faq.html)
1514
- [StackOverflow's `pycord` tag](https://stackoverflow.com/questions/tagged/pycord)
1615

17-
Please try your best not to ask questions in our issue tracker. Most of them don't
18-
belong there unless they provide value to a larger audience.
16+
Please try your best not to ask questions in our issue tracker. Most of them don't belong there
17+
unless they provide value to a larger audience.
1918

2019
## Good Bug Reports
2120

2221
Please be aware of the following things when filing bug reports.
2322

24-
1. Don't open duplicate issues. Please search your issue to see if it has been asked
25-
already. Duplicate issues will be closed.
26-
2. When filing a bug about exceptions or tracebacks, please include the _complete_
27-
traceback. Without the complete traceback the issue might be **unsolvable** and you
28-
will be asked to provide more information.
29-
3. Make sure to provide enough information to make the issue workable. The issue
30-
template will generally walk you through the process but they are enumerated here as
31-
well:
32-
- A **summary** of your bug report. This is generally a quick sentence or two to
33-
describe the issue in human terms.
34-
- Guidance on **how to reproduce the issue**. Ideally, this should have a small code
35-
sample that allows us to run and see the issue for ourselves to debug. **Please
36-
make sure that the token is not displayed**. If you cannot provide a code snippet,
37-
then let us know what the steps were, how often it happens, etc.
23+
1. Don't open duplicate issues. Please search your issue to see if it has been asked already.
24+
Duplicate issues will be closed.
25+
2. When filing a bug about exceptions or tracebacks, please include the _complete_ traceback.
26+
Without the complete traceback the issue might be **unsolvable** and you will be asked to provide
27+
more information.
28+
3. Make sure to provide enough information to make the issue workable. The issue template will
29+
generally walk you through the process but they are enumerated here as well:
30+
- A **summary** of your bug report. This is generally a quick sentence or two to describe the
31+
issue in human terms.
32+
- Guidance on **how to reproduce the issue**. Ideally, this should have a small code sample that
33+
allows us to run and see the issue for ourselves to debug. **Please make sure that the token is
34+
not displayed**. If you cannot provide a code snippet, then let us know what the steps were,
35+
how often it happens, etc.
3836
- Tell us **what you expected to happen**. That way we can meet that expectation.
39-
- Tell us **what actually happens**. What ends up happening in reality? It's not
40-
helpful to say "it fails" or "it doesn't work". Say _how_ it failed, do you get an
41-
exception? Does it hang? How are the expectations different from reality?
42-
- Tell us **information about your environment**. What version of Pycord are you
43-
using? How was it installed? What operating system are you running on? These are
44-
valuable questions and information that we use.
37+
- Tell us **what actually happens**. What ends up happening in reality? It's not helpful to say
38+
"it fails" or "it doesn't work". Say _how_ it failed, do you get an exception? Does it hang?
39+
How are the expectations different from reality?
40+
- Tell us **information about your environment**. What version of Pycord are you using? How was
41+
it installed? What operating system are you running on? These are valuable questions and
42+
information that we use.
4543

46-
If the bug report is missing this information then it'll take us longer to fix the
47-
issue. We will probably ask for clarification, and barring that if no response was given
48-
then the issue will be closed.
44+
If the bug report is missing this information then it'll take us longer to fix the issue. We will
45+
probably ask for clarification, and barring that if no response was given then the issue will be
46+
closed.
4947

5048
## Submitting a Pull Request
5149

52-
Submitting a pull request is fairly simple, just make sure it focuses on a single aspect
53-
and doesn't manage to have scope creep and it's probably good to go. It would be
54-
incredibly lovely if the style is consistent to that found in the project. This project
55-
follows PEP-8 guidelines (mostly) with a column limit of 120.
50+
Submitting a pull request is fairly simple, just make sure it focuses on a single aspect and doesn't
51+
manage to have scope creep and it's probably good to go. It would be incredibly lovely if the style
52+
is consistent to that found in the project. This project follows PEP-8 guidelines (mostly) with a
53+
column limit of 120.
5654

5755
## Use of "type: ignore" comments
5856

59-
In some cases, it might be necessary to ignore type checker warnings for one reason or
60-
another. If that is the case, it is **required** that a comment is left explaining why
61-
you are deciding to ignore type checking warnings.
57+
In some cases, it might be necessary to ignore type checker warnings for one reason or another. If
58+
that is the case, it is **required** that a comment is left explaining why you are deciding to
59+
ignore type checking warnings.
6260

6361
## Licensing
6462

6563
By submitting a pull request, you agree to and accept the following:
6664

6765
1. You hold the copyright on all submitted code inside said pull request.
6866
2. You transfer all rights to the owner of this repository.
69-
3. If you are found to be in fault with any of the above, we shall not be held
70-
responsible in any way after the pull request has been merged.
67+
3. If you are found to be in fault with any of the above, we shall not be held responsible in any
68+
way after the pull request has been merged.
7169

7270
## Git Commit Styling
7371

74-
Not following this guideline could lead to your pull being squashed for a cleaner commit
75-
history
72+
Not following this guideline could lead to your pull being squashed for a cleaner commit history
7673

7774
Some style guides we would recommend using in your pulls:
7875

79-
The [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) style is a
80-
very widely used style and a good style to start with.
76+
The [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) style is a very widely
77+
used style and a good style to start with.
8178

82-
The [gitmoji](https://gitmoji.dev) style guide would make your pull look more lively and
83-
different to others.
79+
The [gitmoji](https://gitmoji.dev) style guide would make your pull look more lively and different
80+
to others.
8481

85-
We don't limit nor deny your pulls when you're using another style although, please make
86-
sure it is appropriate and makes sense in this library.
82+
We don't limit nor deny your pulls when you're using another style although, please make sure it is
83+
appropriate and makes sense in this library.
8784

8885
## AI Usage Policy
8986

90-
The use of LLMs or other AI tools must be disclosed at all times. Disclosure is required
91-
for any interaction or contribution including, but not limited to, the following:
87+
The use of LLMs or other AI tools must be disclosed at all times. Disclosure is required for any
88+
interaction or contribution including, but not limited to, the following:
9289

9390
- Modifying code
9491
- Modifying documentation
9592
- Discussion with maintainers
9693
- Issue bodies and comments
9794
- Pull request bodies and comments
9895

99-
The disclosure must specify the tools used and the scope of their application. The
100-
primary location of disclosure should be in the body of issues and pull requests.
96+
The disclosure must specify the tools used and the scope of their application. The primary location
97+
of disclosure should be in the body of issues and pull requests.
10198

10299
If you are not a native English speaker, using LLMs or other AI tools for translation is
103-
discouraged. It can distort your intended meaning and make a human-written contribution
104-
look AI-generated. Please use a non-LLM-based translation tool such as
105-
[DeepL](https://deepl.com) instead.
100+
discouraged. It can distort your intended meaning and make a human-written contribution look
101+
AI-generated. Please use a non-LLM-based translation tool such as [DeepL](https://deepl.com)
102+
instead.
106103

107-
The use of fully autonomous LLMs or other AI tools to open issues or pull requests is
108-
strictly forbidden.
104+
The use of fully autonomous LLMs or other AI tools to open issues or pull requests is strictly
105+
forbidden.
109106

110-
Failure to follow this policy may result in a temporary or permanent ban from our
111-
organization.
107+
Failure to follow this policy may result in a temporary or permanent ban from our organization.
112108

113-
The use of LLMs or AI tools by new contributors is highly discouraged, and will likely
114-
result in the direct closure of your issue or pull request.
109+
The use of LLMs or AI tools by new contributors is highly discouraged, and will likely result in the
110+
direct closure of your issue or pull request.
115111

116-
Furthermore, as outlined earlier in §"Licensing," by submitting a pull request, you
117-
agree that you hold the copyright on your contribution, and that you understand how your
118-
submitted code works and are able to explain its purpose, behavior, and impact when
119-
requested.
112+
Furthermore, as outlined earlier in §"Licensing," by submitting a pull request, you agree that you
113+
hold the copyright on your contribution, and that you understand how your submitted code works and
114+
are able to explain its purpose, behavior, and impact when requested.

0 commit comments

Comments
 (0)