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

docs: (toy) optional instead of error in parser #1070

Merged
merged 3 commits into from Jun 4, 2023
Merged

Conversation

superlopuh
Copy link
Member

Just a small change to help me debug, only throw errors if something has gone wrong, bool/optional otherwise.

@superlopuh superlopuh added the documentation Improvements or additions to documentation label Jun 4, 2023
@superlopuh superlopuh self-assigned this Jun 4, 2023
@codecov
Copy link

codecov bot commented Jun 4, 2023

Codecov Report

Patch coverage: 77.77% and project coverage change: -0.14 ⚠️

Comparison is base (b2d1bba) 86.93% compared to head (b289d0f) 86.79%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1070      +/-   ##
==========================================
- Coverage   86.93%   86.79%   -0.14%     
==========================================
  Files         129      130       +1     
  Lines       19788    20423     +635     
  Branches     3017     3097      +80     
==========================================
+ Hits        17202    17727     +525     
- Misses       2077     2184     +107     
- Partials      509      512       +3     
Impacted Files Coverage Δ
docs/Toy/toy/frontend/parser.py 78.78% <77.77%> (-3.31%) ⬇️

... and 28 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Collaborator

@math-fehr math-fehr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@superlopuh superlopuh merged commit 83f39a2 into main Jun 4, 2023
12 checks passed
@superlopuh superlopuh deleted the sasha/toy/peek branch June 4, 2023 16:38
cigarichard pushed a commit to cigarichard/xdsl that referenced this pull request Jun 17, 2023
Just a small change to help me debug, only throw errors if something has
gone wrong, bool/optional otherwise.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants