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

⚡ Feature: Better error reporting, messaging #38

Open
wants to merge 28 commits into
base: vardec_varass_dependency
Choose a base branch
from
Open

⚡ Feature: Better error reporting, messaging #38

wants to merge 28 commits into from

Conversation

deavmi
Copy link
Contributor

@deavmi deavmi commented Apr 2, 2024

Please see this.

@deavmi
Coords
7ae0f83 
- New type added

Token

- Added `getCoords()`
@deavmi
Coords
ee73c32 
- Fixed missing import
@deavmi
Statement
4f474a3 
- Made `abstract`
@deavmi
Reporting
f15079b 
- New module added

LineInfo

- Added newtype
@deavmi
Statement
544bb66 
- Added `setLineInfo(LineInfo)` and `getLineInfo()`
@deavmi
Dub
5bea0b5 
- Addd `niknaks` version `0.9.8`
@deavmi
Tokens
aaeda4c 
- Added `setOrigin(string)` and `getOrigin()`

Coords

- Now has a `getLine()` and `getColumn()`
@deavmi
Reporting
cde01f0 
- Added textual reporting mechanism
@deavmi deavmi added documentation Improvements or additions to documentation enhancement New feature or request labels Apr 2, 2024
deavmi added 18 commits April 2, 2024 18:57
@deavmi
Reporting
18955be 
- Moved
- Work on a new more first-principles reporting mechanism is being done
@deavmi
Reporting
64e9c3c 
- Fixed up `LineInfo`
- Token-based reporter uses base method now
@deavmi
Data
cb393ae 
- Disabled that code for now
@deavmi
Statement
8170ce0 
- Now has getter and setter for `LineInfo`
@deavmi
Statement
38eac0a 
- Fixed doc
@deavmi
Tokens
e74fdd3 
- Moved `Coords`

Reporting

- Added Coords
- Cleaned up reporting
@deavmi
Tokens
e87385c 
- Added derivation to `LineInfo`
@deavmi
Reporting
f47ed80 
- Use `Token`-based `LineInfo` direct derivation
@deavmi
Tokens
f2cfa0b 
- Cleaned up

Reporting

- Use new `Token` API
@deavmi
Token
b528bf4 
- Made `getOrigin()` private
@deavmi
Reporting
bfa1295 
- Updated API
@deavmi
SyntaxError
adb6b20 
- The generated message now uses `report(string, Token)` to generate its `msg`
@deavmi
Parser
6382849 
- Cleaned up
@deavmi
Parser
8dfd3f0 
- Added some doodads
- Documented
@deavmi
Parser
ffdd508 
- Fixed typo
@deavmi
Parser
1091f3d 
- Set line information for `WhileLoop`
@deavmi
Merge branch 'vardec_varass_dependency' into feature/reporting
ea6b4a1
@deavmi
Reporting
d1e569a 
- Fixed unittest from not compiling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@deavmi deavmi

@GustavMeyer GustavMeyer

Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@deavmi @GustavMeyer