Skip to content

Commit 74d650e

Browse files
authored
Merge v3.0.0 to main branch
2 parents 626c20c + 0392f12 commit 74d650e

40 files changed

+4171
-1704
lines changed
Lines changed: 48 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,48 @@
1+
### Title of Pull Request
2+
3+
Fixes #[Issue Number]
4+
5+
#### Description
6+
7+
_Briefly describe the changes in this pull request._
8+
9+
- [Summary of major changes]
10+
- [Summary of minor changes]
11+
- [Any other relevant information]
12+
13+
#### Type of Change
14+
15+
Please delete options that are not relevant.
16+
17+
- [ ] Bug fix (non-breaking change which fixes an issue)
18+
- [ ] New feature (non-breaking change which adds functionality)
19+
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
20+
- [ ] This change requires a documentation update
21+
22+
#### How Has This Been Tested?
23+
24+
Please describe the tests that you ran to verify your changes.
25+
Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.
26+
27+
- [ ] Test A
28+
- [ ] Test B
29+
30+
#### Checklist:
31+
32+
- [ ] My code follows the style guidelines of this project
33+
- [ ] I have performed a self-review of my own code
34+
- [ ] I have commented on my code, particularly in hard-to-understand areas
35+
- [ ] I have made corresponding changes to the documentation
36+
- [ ] My changes generate no new warnings
37+
- [ ] I have used Conventional Commits for my commit messages
38+
- [ ] I have added tests that prove my fix is effective or that my feature works
39+
- [ ] New and existing unit tests pass locally with my changes
40+
- [ ] Any dependent changes have been merged and published in downstream modules
41+
42+
#### Screenshots (if applicable)
43+
44+
_Attach screenshots or screen recordings of the changes, if any._
45+
46+
#### Additional Context
47+
48+
_Add any other context or information about the pull request here._

.github/dependabot.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,4 +16,3 @@ updates:
1616
commit-message:
1717
prefix: "poetry"
1818
target-branch: "develop"
19-

.github/workflows/codeql.yml

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,15 @@ name: CodeQL
22

33
on:
44
push:
5-
branches: [ 'main', 'develop' ]
5+
branches:
6+
- main
7+
- develop
8+
69
pull_request:
710
# The branches below must be a subset of the branches above
8-
branches: [ 'main', 'develop' ]
11+
branches:
12+
- main
13+
- develop
914

1015
jobs:
1116
analyze:
@@ -20,8 +25,6 @@ jobs:
2025
fail-fast: false
2126
matrix:
2227
language: [ 'python' ]
23-
# CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python', 'ruby' ]
24-
# Learn more about CodeQL language support at https://aka.ms/codeql-docs/language-support
2528

2629
steps:
2730
- name: Checkout repository
@@ -32,12 +35,6 @@ jobs:
3235
uses: github/codeql-action/init@v2
3336
with:
3437
languages: ${{ matrix.language }}
35-
# If you wish to specify custom queries, you can do so here or in a config file.
36-
# By default, queries listed here will override any specified in a config file.
37-
# Prefix the list here with "+" to use these queries and those in the config file.
38-
39-
# Details on CodeQL's query packs refer to : https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs
40-
# queries: security-extended,security-and-quality
4138

4239
- name: Perform CodeQL Analysis
4340
uses: github/codeql-action/analyze@v2

.github/workflows/greetings.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
name: Greetings
22

3-
on: [pull_request_target, issues]
3+
on: [ pull_request, issues ]
44

55
jobs:
66
greeting:

.github/workflows/tox.yml

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,27 +2,32 @@ name: Tests
22

33
on:
44
push:
5-
branches: [ 'main', 'develop' ]
5+
branches:
6+
- main
7+
- develop
8+
69
pull_request:
710
# The branches below must be a subset of the branches above
8-
branches: [ 'main', 'develop' ]
11+
branches:
12+
- main
13+
- develop
914

1015
jobs:
1116
tests:
1217
runs-on: ubuntu-latest
1318
strategy:
1419
matrix:
15-
python-version: ['3.7', '3.8', '3.9', '3.10']
20+
python-version: [ '3.10', '3.11' ]
1621

1722
steps:
18-
- uses: actions/checkout@v2
23+
- uses: actions/checkout@v3
1924
- name: Set up Python V${{ matrix.python-version }}
20-
uses: actions/setup-python@v2
25+
uses: actions/setup-python@v4
2126
with:
2227
python-version: ${{ matrix.python-version }}
23-
- name: Install dependencies
28+
- name: Install tox-gh-actions
2429
run: |
2530
python -m pip install --upgrade pip
2631
pip install tox tox-gh-actions
27-
- name: Test w/ Tox
28-
run: tox
32+
- name: Tests with Tox
33+
run: tox

.gitignore

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,3 +36,9 @@ mac_cleanup/modules.toml
3636

3737
# tox
3838
.tox/*
39+
40+
# ruff
41+
.ruff_cache/
42+
43+
# macOS stuff
44+
.DS_Store

.pre-commit-config.yaml

Lines changed: 55 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,55 @@
1+
default_stages:
2+
- push
3+
4+
repos:
5+
- repo: https://github.com/commitizen-tools/commitizen
6+
rev: v2.42.1
7+
hooks:
8+
- id: commitizen
9+
name: Check Convenient Commits
10+
stages: [ commit-msg ]
11+
12+
- repo: https://github.com/psf/black
13+
rev: 23.3.0
14+
hooks:
15+
- id: black
16+
name: Run black
17+
types: [ python ]
18+
19+
- repo: https://github.com/pycqa/isort
20+
rev: 5.12.0
21+
hooks:
22+
- id: isort
23+
name: Run isort
24+
types: [ python ]
25+
26+
- repo: https://github.com/PyCQA/docformatter
27+
rev: v1.5.1
28+
hooks:
29+
- id: docformatter
30+
name: Run docformatter
31+
types: [ python ]
32+
args: [ --config=./pyproject.toml]
33+
34+
- repo: local
35+
hooks:
36+
- id: pytest
37+
name: Run pytest
38+
language: system
39+
pass_filenames: false
40+
types: [ python ]
41+
entry: poetry run pytest
42+
43+
- id: pyright
44+
name: Run pyright
45+
language: system
46+
pass_filenames: false
47+
types: [ python ]
48+
entry: poetry run pyright
49+
50+
- id: ruff
51+
name: Run ruff
52+
language: system
53+
pass_filenames: false
54+
types: [ python ]
55+
entry: poetry run ruff check .

CODE_OF_CONDUCT.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
# TL;DR
44

5-
> ### Be nice to each other or you'll face the consequences
5+
> ### Be nice to each other, or you'll face the consequences
66
77
## Our Pledge
88

@@ -129,4 +129,4 @@ enforcement ladder](https://github.com/mozilla/diversity).
129129

130130
For answers to common questions about this code of conduct, see the FAQ at
131131
https://www.contributor-covenant.org/faq. Translations are available at
132-
https://www.contributor-covenant.org/translations.
132+
https://www.contributor-covenant.org/translations.

CONTRIBUTING.md

Lines changed: 23 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,14 @@ Read repo's [Code of Conduct](./CODE_OF_CONDUCT.md) to keep community respectabl
77
## TL;DR
88

99
> #### 1. [Fork](https://docs.github.com/en/github/getting-started-with-github/fork-a-repo#fork-an-example-repository) the repo
10-
> #### 2. [Checkout](https://www.atlassian.com/git/tutorials/using-branches/git-checkout) from 'develop' branch
11-
> #### 3. [Commit](https://www.atlassian.com/git/tutorials/saving-changes/git-commit) your changes
12-
> #### 4. [Push the changes](https://docs.github.com/en/get-started/using-git/pushing-commits-to-a-remote-repository#about-git-push) to your fork
13-
> #### 4. [Open PR](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request) to 'develop' branch
14-
> #### Congrats :tada::sparkles:. CYA in your PR's comments 😉
10+
> #### 2. [Checkout](https://www.atlassian.com/git/tutorials/using-branches/git-checkout) from `develop` branch
11+
> #### 3. Install dependencies: `pip install poetry && poetry install`
12+
> #### 3. Setup `pre-commit` hooks: `pre-commit install --hook-type pre-commit --hook-type pre-push`
13+
> #### 4. [Commit](https://www.atlassian.com/git/tutorials/saving-changes/git-commit) your changes with [Conventional Commits](https://www.conventionalcommits.org)
14+
> #### 5. Run tests: `poetry run tox`
15+
> #### 6. [Push the changes](https://docs.github.com/en/get-started/using-git/pushing-commits-to-a-remote-repository#about-git-push) to your fork
16+
> #### 7. [Open PR](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request) to 'develop' branch
17+
> #### Congrats :tada::sparkles:. CYA in your PRs 😉
1518
1619

1720
## New contributor guide
@@ -42,10 +45,20 @@ If you find an issue to work on, just post a comment on the issue's page and you
4245

4346
### Make Changes
4447

45-
1. Fork the repository.
46-
- [Fork the repo](https://docs.github.com/en/github/getting-started-with-github/fork-a-repo#fork-an-example-repository) so that you can make your changes without affecting the original project until you're ready to merge them.
47-
48-
2. Create a working branch out of 'develop' and start with your changes!
48+
1. [Fork the repository](https://docs.github.com/en/github/getting-started-with-github/fork-a-repo#fork-an-example-repository) so that you can make your changes without affecting the original project until you're ready to merge them.
49+
2. Create a working branch out of 'develop' and start with your changes
50+
3. Install dependencies
51+
```bash
52+
pip install poetry && poetry install
53+
```
54+
4. Setup pre-commit hooks
55+
```bash
56+
pre-commit install --hook-type pre-commit --hook-type pre-push
57+
```
58+
5. Run tests before pushing
59+
```bash
60+
poetry run tox
61+
```
4962

5063
### Commit your update
5164

@@ -68,4 +81,4 @@ Once you submit your PR, someone of collaborators will come and see your PR.
6881
6982
Congratulations :tada::sparkles:
7083
71-
Once your PR is merged, your contributions will be publicly visible on the [mac-cleanup-py](https://github.com/mac-cleanup/mac-cleanup-py).
84+
Once your PR is merged, your contributions will be publicly visible on the [mac-cleanup-py](https://github.com/mac-cleanup/mac-cleanup-py).

0 commit comments

Comments
 (0)