Skip to content

Commit

Permalink
Update 'dev' target
Browse files Browse the repository at this point in the history
  • Loading branch information
jacebrowning committed Mar 11, 2022
1 parent 7caaec9 commit 7dad721
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 15 deletions.
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Expand Up @@ -51,7 +51,7 @@ $ make docs
Keep all of the above tasks running on change:

```sh
$ make watch
$ make dev
```

> In order to have OS X notifications, `brew install terminal-notifier`.
Expand Down
4 changes: 2 additions & 2 deletions Makefile
Expand Up @@ -9,8 +9,8 @@ all: install
.PHONY: ci
ci: format check test mkdocs ## Run all tasks that determine CI status

.PHONY: watch
watch: install .clean-test ## Continuously run all CI tasks when files chanage
.PHONY: dev
dev: install .clean-test ## Continuously run all CI tasks when files chanage
poetry run sniffer

.PHONY: demo
Expand Down
2 changes: 1 addition & 1 deletion gitman/models/config.py
Expand Up @@ -293,7 +293,7 @@ def get_dependencies(self, depth=None, allow_dirty=True):
common.newline()
common.indent()

for source in self.sources:
for source in self._get_sources(use_locked=False):

if depth == 0:
log.info("Skipped dependency: %s", source.name)
Expand Down
22 changes: 11 additions & 11 deletions scent.py
Expand Up @@ -22,29 +22,29 @@ class Options:
rerun_args = None

targets = [
(('make', 'test-unit', 'DISABLE_COVERAGE=true'), "Unit Tests", True),
(('make', 'test-all'), "Integration Tests", False),
(('make', 'check'), "Static Analysis", True),
(('make', 'docs'), None, True),
(("make", "test-unit", "DISABLE_COVERAGE=true"), "Unit Tests", True),
(("make", "test-all"), "Integration Tests", False),
(("make", "check"), "Static Analysis", True),
(("make", "mkdocs"), None, True),
]


@select_runnable('run_targets')
@select_runnable("run_targets")
@file_validator
def python_files(filename):
return filename.endswith('.py') and '.py.' not in filename
return filename.endswith(".py") and ".py." not in filename


@select_runnable('run_targets')
@select_runnable("run_targets")
@file_validator
def html_files(filename):
return filename.split('.')[-1] in ['html', 'css', 'js']
return filename.split(".")[-1] in ["html", "css", "js"]


@runnable
def run_targets(*args):
"""Run targets for Python."""
Options.show_coverage = 'coverage' in args
Options.show_coverage = "coverage" in args

count = 0
for count, (command, title, retry) in enumerate(Options.targets, start=1):
Expand Down Expand Up @@ -74,7 +74,7 @@ def call(command, title, retry):
return False

print("")
print("$ %s" % ' '.join(command))
print("$ %s" % " ".join(command))
failure = subprocess.call(command)

if failure and retry:
Expand All @@ -92,6 +92,6 @@ def show_notification(message, title):
def show_coverage():
"""Launch the coverage report."""
if Options.show_coverage:
subprocess.call(['make', 'read-coverage'])
subprocess.call(["make", "read-coverage"])

Options.show_coverage = False

0 comments on commit 7dad721

Please sign in to comment.