Skip to content

Print concise error messages when invoking yaml2ics command#55

Merged
jarrodmillman merged 1 commit into
mainfrom
main-invoke
Jun 13, 2023
Merged

Print concise error messages when invoking yaml2ics command#55
jarrodmillman merged 1 commit into
mainfrom
main-invoke

Conversation

@stefanv

@stefanv stefanv commented Jun 8, 2023

Copy link
Copy Markdown
Member

This suppresses the traceback currently printed when running yaml2ics.

yaml2ics can be run in three ways:

- `python yaml2ics.py`
- `python -m yaml2ics`
- `yaml2ics`

This ensures that none of those print tracebacks, while testing (which
relies on errors being raised) still functions as expected.
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #55 (2cd6270) into main (67d8691) will decrease coverage by 4.81%.
The diff coverage is 100.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@             Coverage Diff             @@
##              main      #55      +/-   ##
===========================================
- Coverage   100.00%   95.19%   -4.81%     
===========================================
  Files            1        1              
  Lines           98      104       +6     
===========================================
+ Hits            98       99       +1     
- Misses           0        5       +5     
Impacted Files Coverage Δ
yaml2ics.py 95.19% <100.00%> (-4.81%) ⬇️

@jarrodmillman jarrodmillman added this to the 0.3 milestone Jun 8, 2023
@jarrodmillman jarrodmillman merged commit f1d53ec into main Jun 13, 2023
@jarrodmillman jarrodmillman deleted the main-invoke branch June 13, 2023 03:54
@jarrodmillman jarrodmillman added the type: Enhancement New feature or request label Jul 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: Enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants