Skip to content

[Merged by Bors] - chore: move #guard_msgs in PosDef to test file#10827

Closed
adomani wants to merge 1 commit intomasterfrom
adomani/lint_guard_msgs
Closed

[Merged by Bors] - chore: move #guard_msgs in PosDef to test file#10827
adomani wants to merge 1 commit intomasterfrom
adomani/lint_guard_msgs

Conversation

@adomani
Copy link
Copy Markdown
Contributor

@adomani adomani commented Feb 21, 2024

This PR moves a #guard_msgs command from LinearAlgebra/Matrix/PosDef to a test file: it is a preparation for #10809.


Open in Gitpod

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Feb 22, 2024

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Feb 22, 2024
mathlib-bors bot pushed a commit that referenced this pull request Feb 22, 2024
This PR moves a `#guard_msgs` command from `LinearAlgebra/Matrix/PosDef` to a test file: it is a preparation for #10809.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Feb 22, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: move #guard_msgs in PosDef to test file [Merged by Bors] - chore: move #guard_msgs in PosDef to test file Feb 22, 2024
@mathlib-bors mathlib-bors bot closed this Feb 22, 2024
@mathlib-bors mathlib-bors bot deleted the adomani/lint_guard_msgs branch February 22, 2024 03:43
mathlib-bors bot pushed a commit that referenced this pull request Feb 22, 2024
This PR introduces an `awk`-based linter for checking that `Mathlib/*.lean` files do not contain `#`-commands.

It takes the names of the commands that begin with `#` by running
```lean
import Mathlib
#help command
```
and after that it is all text-based substitutions.

It triggers an error on lines that begin with `#cmd`, omitting (most) lines contained comment blocks.

CI for this PR should fail until #10772 and #10827 have been merged.
thorimur pushed a commit that referenced this pull request Feb 24, 2024
This PR moves a `#guard_msgs` command from `LinearAlgebra/Matrix/PosDef` to a test file: it is a preparation for #10809.
thorimur pushed a commit that referenced this pull request Feb 24, 2024
This PR introduces an `awk`-based linter for checking that `Mathlib/*.lean` files do not contain `#`-commands.

It takes the names of the commands that begin with `#` by running
```lean
import Mathlib
#help command
```
and after that it is all text-based substitutions.

It triggers an error on lines that begin with `#cmd`, omitting (most) lines contained comment blocks.

CI for this PR should fail until #10772 and #10827 have been merged.
thorimur pushed a commit that referenced this pull request Feb 26, 2024
This PR moves a `#guard_msgs` command from `LinearAlgebra/Matrix/PosDef` to a test file: it is a preparation for #10809.
thorimur pushed a commit that referenced this pull request Feb 26, 2024
This PR introduces an `awk`-based linter for checking that `Mathlib/*.lean` files do not contain `#`-commands.

It takes the names of the commands that begin with `#` by running
```lean
import Mathlib
#help command
```
and after that it is all text-based substitutions.

It triggers an error on lines that begin with `#cmd`, omitting (most) lines contained comment blocks.

CI for this PR should fail until #10772 and #10827 have been merged.
adomani added a commit that referenced this pull request Feb 27, 2024
This PR moves a `#guard_msgs` command from `LinearAlgebra/Matrix/PosDef` to a test file: it is a preparation for #10809.
riccardobrasca pushed a commit that referenced this pull request Mar 1, 2024
This PR moves a `#guard_msgs` command from `LinearAlgebra/Matrix/PosDef` to a test file: it is a preparation for #10809.
riccardobrasca pushed a commit that referenced this pull request Mar 1, 2024
This PR introduces an `awk`-based linter for checking that `Mathlib/*.lean` files do not contain `#`-commands.

It takes the names of the commands that begin with `#` by running
```lean
import Mathlib
#help command
```
and after that it is all text-based substitutions.

It triggers an error on lines that begin with `#cmd`, omitting (most) lines contained comment blocks.

CI for this PR should fail until #10772 and #10827 have been merged.
dagurtomas pushed a commit that referenced this pull request Mar 22, 2024
This PR moves a `#guard_msgs` command from `LinearAlgebra/Matrix/PosDef` to a test file: it is a preparation for #10809.
dagurtomas pushed a commit that referenced this pull request Mar 22, 2024
This PR introduces an `awk`-based linter for checking that `Mathlib/*.lean` files do not contain `#`-commands.

It takes the names of the commands that begin with `#` by running
```lean
import Mathlib
#help command
```
and after that it is all text-based substitutions.

It triggers an error on lines that begin with `#cmd`, omitting (most) lines contained comment blocks.

CI for this PR should fail until #10772 and #10827 have been merged.
xgenereux pushed a commit that referenced this pull request Apr 15, 2024
This PR moves a `#guard_msgs` command from `LinearAlgebra/Matrix/PosDef` to a test file: it is a preparation for #10809.
xgenereux pushed a commit that referenced this pull request Apr 15, 2024
This PR introduces an `awk`-based linter for checking that `Mathlib/*.lean` files do not contain `#`-commands.

It takes the names of the commands that begin with `#` by running
```lean
import Mathlib
#help command
```
and after that it is all text-based substitutions.

It triggers an error on lines that begin with `#cmd`, omitting (most) lines contained comment blocks.

CI for this PR should fail until #10772 and #10827 have been merged.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants