Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: use 'lake test' #787

Merged
merged 2 commits into from May 13, 2024
Merged

feat: use 'lake test' #787

merged 2 commits into from May 13, 2024

Conversation

semorrison
Copy link
Collaborator

@semorrison semorrison commented May 9, 2024

This is an attempt to switch to using lake test. Gradually more of this functionality will be taken over by lake, but I would like to push this along and see if we can't get some of the major repositories using lake test.

@semorrison semorrison added the awaiting-review This PR is ready for review; the author thinks it is ready to be merged. label May 9, 2024
@semorrison semorrison requested a review from digama0 May 9, 2024 06:24
GNUmakefile Show resolved Hide resolved
GNUmakefile Show resolved Hide resolved
@semorrison semorrison added the will-merge-soon PR will be merged soon. Any concerns should be raised quickly. label May 12, 2024
@semorrison semorrison merged commit 14f2585 into main May 13, 2024
3 checks passed
@semorrison semorrison deleted the lake_test branch May 13, 2024 08:01
mathlib-bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request May 20, 2024
In an effort to start moving to a more Lean+Lake based infrastructure, this installs a test runner which is a thin wrapper around Batteries' test runner introduced in leanprover-community/batteries#787.

The `liskin/gh-problem-matcher-wrap@v3` problem matcher was already broken for the test suite, and will remain broken here. There's a tracking issue at #12946. We can restore that once the new test behaviour has converged.

- [x] depends on: leanprover-community/batteries#797
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review This PR is ready for review; the author thinks it is ready to be merged. will-merge-soon PR will be merged soon. Any concerns should be raised quickly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants