Skip to content

Commit

Permalink
Don't ignore undeclared-scope warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
anton-trunov committed Jan 31, 2020
1 parent 73303f9 commit 1a21e62
Show file tree
Hide file tree
Showing 7 changed files with 10 additions and 18 deletions.
10 changes: 2 additions & 8 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,21 +42,15 @@ matrix:
- env:
- COQ=8.10
<<: *NIX
- env:
- COQ=8.9
<<: *NIX
- env:
- COQ=8.8
<<: *NIX

# Test supported versions of Coq via OPAM
- env:
- COQ_IMAGE=coqorg/coq:dev
- COQ_IMAGE=coqorg/coq:8.11
- PACKAGE=coq-lemma-overloading.dev
- NJOBS=2
<<: *OPAM
- env:
- COQ_IMAGE=coqorg/coq:8.11
- COQ_IMAGE=coqorg/coq:dev
- PACKAGE=coq-lemma-overloading.dev
- NJOBS=2
<<: *OPAM
Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,8 @@ re-implementations for comparison.
- Anton Trunov ([**@anton-trunov**](https://github.com/anton-trunov))
- Karl Palmskog ([**@palmskog**](https://github.com/palmskog))
- License: [GNU General Public License v3.0 or later](LICENSE.md)
- Compatible Coq versions: 8.8 or later (use releases for other Coq versions)
- Additional Coq dependencies:
- Compatible Coq versions: 8.10 or later (use releases for other Coq versions)
- Additional dependencies:
- [MathComp](https://math-comp.github.io) 1.7.0 or later (`ssreflect` suffices)
- Coq namespace: `LemmaOverloading`
- Related publication(s):
Expand Down
1 change: 0 additions & 1 deletion _CoqProject
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
-arg -w -arg -notation-overridden
-arg -w -arg -projection-no-head-constant
-arg -w -arg -redundant-canonical-projection
-arg -w -arg -undeclared-scope

theories/prelude.v
theories/rels.v
Expand Down
3 changes: 1 addition & 2 deletions coq-lemma-overloading.opam
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,7 @@ re-implementations for comparison."""
build: [make "-j%{jobs}%"]
install: [make "install"]
depends: [
"ocaml"
"coq" {(>= "8.8" & < "8.12~") | (= "dev")}
"coq" {(>= "8.10" & < "8.12~") | (= "dev")}
"coq-mathcomp-ssreflect" {(>= "1.7" & < "1.11~") | (= "dev")}
]

Expand Down
8 changes: 3 additions & 5 deletions meta.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,17 +55,15 @@ license:
file: LICENSE.md

supported_coq_versions:
text: 8.8 or later (use releases for other Coq versions)
opam: '{(>= "8.8" & < "8.12~") | (= "dev")}'
text: 8.10 or later (use releases for other Coq versions)
opam: '{(>= "8.10" & < "8.12~") | (= "dev")}'

tested_coq_nix_versions:
- version_or_url: '8.10'
- version_or_url: '8.9'
- version_or_url: '8.8'

tested_coq_opam_versions:
- version: dev
- version: '8.11'
- version: dev

dependencies:
- opam:
Expand Down
1 change: 1 addition & 0 deletions theories/rels.v
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ Lemma orpT p : p \/ True <-> True. Proof. by intuition. Qed.
Lemma orFp p : False \/ p <-> p. Proof. by intuition. Qed.
Lemma orpF p : p \/ False <-> p. Proof. by intuition. Qed.

Declare Scope rel_scope.
Delimit Scope rel_scope with rel.
Open Scope rel_scope.

Expand Down
1 change: 1 addition & 0 deletions theories/stsep.v
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ Set Implicit Arguments.
Unset Strict Implicit.
Unset Printing Implicit Defensive.

Declare Scope stsep_scope.
Delimit Scope stsep_scope with stsep.
Open Scope stsep_scope.

Expand Down

0 comments on commit 1a21e62

Please sign in to comment.