Browse Source

module: add Makefile for development commands

master
cn 1 month ago
parent
commit
ded2908236
3 changed files with 30 additions and 2 deletions
  1. +2
    -2
      .travis.yml
  2. +24
    -0
      Makefile
  3. +4
    -0
      README.md

+ 2
- 2
.travis.yml View File

@@ -13,5 +13,5 @@ install:
- poetry install

script:
- poetry run pylint veml6070
- poetry run pytest
- make lint
- make test

+ 24
- 0
Makefile View File

@@ -0,0 +1,24 @@

# https://tech.davis-hansson.com/p/make/
SHELL := bash
.ONESHELL:
.SHELLFLAGS := -eu -o pipefail -c
.DELETE_ON_ERROR:
.DEFAULT_GOAL := all
MAKEFLAGS += --warn-undefined-variables
MAKEFLAGS += --no-builtin-rules

.PHONY: all
all: lint test ## Run lint and test (default goal)

.PHONY: lint
lint: ## Lint all source code
poetry run pylint veml6070

.PHONY: test
test: ## Run all tests
poetry run pytest

.PHONY: help
help: ## Print this help text
@grep -E '^[a-zA-Z_-]+:.*?## .*$$' $(MAKEFILE_LIST) | awk 'BEGIN {FS = ":.*?## "}; {printf "\033[36m%-15s\033[0m %s\n", $$1, $$2}'

+ 4
- 0
README.md View File

@@ -21,6 +21,10 @@ In September 2019 it was discovered (and fixed) that:
- previously the sensor was never shutdown between measurements which wastes power but still takes measurements successfully
- the UVA light intensity was calculated wrongly (too high) for `rset != RSET_240K` due to wrong compensation: higher `rset` leads to higher sampling time leads to higher absolute ADC step counts which *should* lead to every ADC step indicating a smaller amount of `W/(m*m)` of UVA power and a higher precision of the final UVA power but it wrongly behaved the opposite way. The `integration_time` worked correctly all the time.

## Develop

Run `make help` to find out about the available development commands.

## License

Python files in this repository are released under the [MIT license](LICENSE).

Loading…
Cancel
Save