Skip to content

Commit

Permalink
Adds coverage reporting (#44)
Browse files Browse the repository at this point in the history
* Removing 2.7 support and switching everything to new practices

* New package setup

* Update logger

* Logger update and removed six calls

* Updating format and removing eval statements

* Adding pathlib and making sure TokenNumber and Token are taken in load function

* Working on typing hints to improve readability

* Typing hints

* Parameterize explicit solve test

* Parameterize defaults tests

* Parameterize implicit solving test

* Parameterize parsing tests

* Parameterize repeated tokens

* Parameterize repeating tokens

* Parameterization of anchor tests

* More parameterization on naming tests

* Remove comment

* WIP rules tests parameterization

* Validate basic rule creation parameters

* Rules tests parametrization

* Wip token tests parametrization

* Tokens tests parametrization finished

* Update docs

* Fixing outdated tox

* Remove old travis

* Setup shell before invoking ruff

* Using pipenv run

* Badges

* Fixing docs buils

* docs requirements

* Update yml with requirements

* Update badges in docs

* Update coverage

* Adds coverage badge

* Testing environment

* Add lcov report

* wrong token placement
  • Loading branch information
xiancg authored Aug 10, 2024
1 parent 6f4365a commit 894f900
Show file tree
Hide file tree
Showing 6 changed files with 56 additions and 105 deletions.
9 changes: 8 additions & 1 deletion .github/workflows/ci_vfxnaming.yml
Original file line number Diff line number Diff line change
@@ -1,11 +1,13 @@
name: VFXNaming CI

on: [push]
on: [push, pull_request]

jobs:
build:

runs-on: ubuntu-latest
environment:
name: coverage
strategy:
fail-fast: false
matrix:
Expand All @@ -30,3 +32,8 @@ jobs:
- name: Run tests with tox
run: |
tox -e py310ci
- name: Coveralls
uses: coverallsapp/github-action@v2
with:
github-token: ${{ secrets.GITHUB_TOKEN }}

2 changes: 1 addition & 1 deletion Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ pytest = "*"
pytest-cov = "*"
pytest-datafiles = "*"
ruff = "*"
python-coveralls = "*"
coveralls = "*"
sphinx = "*"
sphinx-rtd-theme = "*"
twine = "*"
Expand Down
143 changes: 42 additions & 101 deletions Pipfile.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ VFX Naming Conventions Framework

.. image:: https://github.com/xiancg/vfxnaming/actions/workflows/ci_vfxnaming.yml/badge.svg?event=push&branch=master
.. image:: https://readthedocs.org/projects/naming-conventions/badge/?version=latest
.. image:: https://coveralls.io/repos/github/xiancg/vfxnaming/badge.svg
.. image:: https://img.shields.io/github/license/readthedocs/actions

A complete suite of tools to manage naming conventions from one or more "Rule repositories". Structure naming rules with your own custom tokens. Then use the library to solve names following those rules so your naming is consistent, and also to parse metadata from exisiting names (cus a name is basically a collection of metadata, right?)
Expand Down
2 changes: 2 additions & 0 deletions docs/source/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@ VFX Naming Conventions Docs
.. image:: https://github.com/xiancg/vfxnaming/actions/workflows/ci_vfxnaming.yml/badge.svg?event=push&branch=master
.. image:: https://readthedocs.org/projects/naming-conventions/badge/?version=latest
.. image:: https://img.shields.io/github/license/readthedocs/actions
.. image:: https://coveralls.io/repos/github/xiancg/vfxnaming/badge.svg


Installation
------------
Expand Down
4 changes: 2 additions & 2 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,8 @@ commands=
deps = pytest
pytest-cov
pytest-datafiles
python-coveralls
commands= pytest -c tox.ini --cov-report term-missing --cov=vfxnaming tests/
coveralls
commands= pytest -c tox.ini --cov-report lcov --cov=vfxnaming tests/

[pytest]
addopts = --maxfail=10 -rf -s
Expand Down

0 comments on commit 894f900

Please sign in to comment.