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.

Sign up for GitHub

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

Jump to bottom

Reerror #2480

Merged
merged 47 commits into from
Oct 29, 2019
Merged

Reerror #2480

merged 47 commits into from
Oct 29, 2019

Conversation

anmonteiro
Copy link
Member

@anmonteiro anmonteiro commented Oct 29, 2019 •
edited
Loading

This is #2439 rebased on current master

let-def and others added 30 commits October 28, 2019 21:24
@let-def @anmonteiro
split reason_toolchain in a few sub-modules
9efc1e8
@let-def @anmonteiro
Remove menhir error messages
198341a
@let-def @anmonteiro
removed more menhir-related error management
51beb3e
@let-def @anmonteiro
refactor errors in semantic actions
ba5106a
@let-def @anmonteiro
REMOVE ME: some notes
c35b355
@let-def @anmonteiro
move more error management code to Reason_errors
8c9a622
@let-def @anmonteiro
recovery annotations
3e40703
@let-def @anmonteiro
add recovery tool
eb5b2bd
@let-def @anmonteiro
build recovery
90e73cb
@let-def @anmonteiro
WIP lexer -> declarative_lexer, remove state
04c0d17
@let-def @anmonteiro
rewrite lexer to isolate state
e62d0e6
@let-def @anmonteiro
factor the deterministic parser out of reason_toolchain
2f18f42
@let-def @anmonteiro
implement support for POSTFIX
4cb2829
@let-def @anmonteiro
introduce reason-multi-parser
7066dec
@let-def @anmonteiro
Switch on recovery
b025634
@let-def @anmonteiro
add TODO plan
6c76635
@let-def @anmonteiro
oops, better to recover more than one token
b754d0e
@let-def @anmonteiro
fix invalid docstrings merging
981acf3
@let-def @anmonteiro
fix DOCSTRING parsing
3504ede
@let-def @anmonteiro
fishy printing
4f85ff4
@let-def @anmonteiro
recovery is now optional
f2871a1
@let-def @anmonteiro
update TODO
494e5c1
@let-def @anmonteiro
attach errors for intf and impl
ea01c39
@let-def @anmonteiro
Implement slightly better error messages
a283f96
@let-def @anmonteiro
Update reason.opam
153aed5
@let-def @anmonteiro
handle EOF
4881250
@anmonteiro
Use Migrate_parsetree.Ast_404 instead of Ast_404
d9de6c3 
Modules are being namespaced, direct access is now deprecated and will
be removed (at some unknown point in the future).
@anmonteiro
Reenable previous error messages (through Reason_parser_explain)
b6b7fe4
@anmonteiro
detect unclosed parentheses
99f5466
@let-def @anmonteiro
TEMPORARY: tweak unclosed parenthesis error message
3c4cf59 
Still broken, the analysis can be improved
let-def and others added 17 commits October 28, 2019 21:24
@let-def @anmonteiro
TEMPORARY: disable JSX error messages on recovery
03d1f68 
Recovery generates empty closing tag
@anmonteiro
wip: distinguish recovery errors
8d0c8ea
@let-def @anmonteiro
insert extension nodes only in recovery mode
80a1db1
@anmonteiro
build with 4.02
6846307
@anmonteiro
instrument reason-error message printer
a950603
@anmonteiro
raise fatal errors if not using Reason_config.recoverable
0b59038
@anmonteiro
no need for lexer_report_error
9fc45f2
@anmonteiro
fix reporting of errors
fa5ff5a
@anmonteiro
refactor version-dependent definitions
653c705
@anmonteiro
move error manipulation to reason_syntax_util
5c315bc
@anmonteiro
menhir-recover: hide warnings about recover.cost attribute on production
d5b2fe9
@let-def @anmonteiro
fix error reporting (pre 4.08)
fae2d66
@let-def @anmonteiro
update more tests
18de472
@let-def @anmonteiro
Abstract version-dependent compiler-libs definitions to ocaml_util.ml
0ba72f8 
Fix build with all versions of OCaml
@let-def @anmonteiro
update tests
b94c105
@anmonteiro
add fix dependency to esy
d930c65
@anmonteiro
fix tests
551d299
@jordwalke jordwalke merged commit b555665 into reasonml:master Oct 29, 2019
This was referenced Dec 1, 2019
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@anmonteiro @jordwalke @let-def