Skip to content

Commit

Permalink
[TESTS] Posible fix for multiple minor issues (- WIP #46 & #49 -)
Browse files Browse the repository at this point in the history
Changes in file .codecov.yml:
 removeed `setup.py` from coverage quota

Changes in file .coveragerc:
 removeed `setup.py` from coverage quota

Changes in file .github/labeler.yml:
 added license to github label patterns

Changes in file .github/workflows/Tests.yml:
 just a version update

Changes in file README.md:
 improved spacing a little

REMOVED file tests/check_codecov_regression
  • Loading branch information
reactive-firewall committed Aug 28, 2024
1 parent e0ac860 commit 418895b
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 177 deletions.
1 change: 0 additions & 1 deletion .codecov.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ flags:
multicast:
paths:
- "multicast/"
- "!tests/"
tests:
paths:
- "tests/"
Expand Down
2 changes: 1 addition & 1 deletion .coveragerc
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ parallel = True
sigterm = True

[report]
include = multicast/*,tests/*,setup.py
include = multicast/*,tests/*
# Regexes for lines to exclude from consideration
exclude_lines =
# Have to re-enable the standard pragma
Expand Down
2 changes: 2 additions & 0 deletions .github/labeler.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,3 +44,5 @@ Github:
- .github/**/*
- .github/*
- README.md
- LICENCE
- LICENCE.md
4 changes: 2 additions & 2 deletions .github/workflows/Tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -435,8 +435,8 @@ jobs:
hash -p ./.github/tool_shlock_helper.sh shlock ;
./tests/check_integration_coverage || echo "::warning file=tests/check_integration_coverage,line=1,endLine=1,title=SKIPPED::SKIP Integration Tests." ;
shell: bash
- name: Upload Python ${{ matrix.python-version }} coverage to Codecov
uses: codecov/codecov-action@v3
- name: Upload Python ${{ matrix.python-version }} integration coverage to Codecov
uses: codecov/codecov-action@v4
with:
token: ${{ secrets.CODECOV_TOKEN }}
files: ./test-reports/coverage_supplement.xml
Expand Down
6 changes: 6 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -140,16 +140,22 @@ p.join() # if not already handled don't forget to join the process and other ove
#### Python function return code meanings

`0` is the default and implies *success*, and means the process has essentially (or actually) returned nothing (or `None`)

`1` is used when a *single* result is returned (caveat: functions may return a single `tuple` instead of `None` to indicate exit code `1` by returning a `boolean` success value, and result (which may also be encapsulated as an iteratable if needed) )

`2` is used to indicate a *value and reason* are returned (caveat: functions may return a single `tuple` with a single value and reason and the value can be a `tuple`)

`-1` is used to mean *many* of unspecified length and otherwise functions as `1`

#### CLI exit code meanings

`0` *success*

`1` *none-sucsess* - and is often accompanied by warnings or errors

`2 >` *failure* of specific reason


#### Everything Else
_(extra exit code meanings)_

Expand Down
173 changes: 0 additions & 173 deletions tests/check_codecov_regression

This file was deleted.

0 comments on commit 418895b

Please sign in to comment.