Skip to content

Name no way to skip the tests - #32

Merged
estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:chore/no-skip-tests-option-the-chain-refuses
Aug 31, 2026
Merged

estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:chore/no-skip-tests-option-the-chain-refuses

Conversation

@estebanzimanyi

Copy link
Copy Markdown
Member

tools/refresh-from-master.sh forwards to MEOS-API's refresh-binding.sh, which
carries no --skip-tests: passing it is an error, not a faster refresh, and
GENERATION.md offered it as a usage example. refresh.conf here names no such
variable.

The README's build recipe drops -DskipTests too, so nothing in the repository
shows a reader how to get an artifact without running the suite that says the
artifact answers.

A full tools/refresh-from-master.sh runs green: 7 binding and 4 benchmark
tests, 0 skipped.

`tools/refresh-from-master.sh` forwards to MEOS-API's `refresh-binding.sh`, which
carries no `--skip-tests`: passing it is an error, not a faster refresh, and
GENERATION.md offered it as a usage example. `refresh.conf` here names no such
variable.

The README's build recipe drops `-DskipTests` too, so nothing in the repository
shows a reader how to get an artifact without running the suite that says the
artifact answers.

A full `tools/refresh-from-master.sh` runs green: 7 binding and 4 benchmark
tests, 0 skipped.
@estebanzimanyi
estebanzimanyi merged commit f2853cf into MobilityDB:main Aug 31, 2026
1 check passed
@estebanzimanyi
estebanzimanyi deleted the chore/no-skip-tests-option-the-chain-refuses branch August 31, 2026 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant