Skip to content

Conversation

@yaphott
Copy link
Contributor

@yaphott yaphott commented Aug 2, 2023

  • Pipeline components
  • Readme generation
  • Improved code structure
  • Corrected version requirement

yaphott added 15 commits March 9, 2023 21:06
Moved helpers into `utils.py`.
Moved constants into `constants.py`.
Added `enums.py`.
Rename `char_as_exp` to simply `escape`, as it is a more accurate name.
Now sharing pattern building functions for both RE and RE2 to be specified by an enum (this could change in the future).
Naming that involves "codepoint" will now be used as "cpoint" instead to save line room.
Added tests.
Updated readme.
Further implementation of subtests.
Edited docstrings and correct some example parts that misrender in the readme generator.
Update Makefile and add cli args to doc generating script.
Added a couple tests and improved test method names.
- Updated Makefile to be more informative/consistent.
- Prepare for codecov in readme generator config, and correct main branch name.
- Insert badges for codecov into install.md.jinja template.
- More aggressive path resolving in readme generator.
@yaphott yaphott requested a review from rtstacy August 2, 2023 03:00
@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

❗ No coverage uploaded for pull request base (main@d745782). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage        ?   82.47%           
=======================================
  Files           ?        5           
  Lines           ?       97           
  Branches        ?       12           
=======================================
  Hits            ?       80           
  Misses          ?       16           
  Partials        ?        1           
Flag Coverage Δ
unittests 82.47% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yaphott yaphott merged commit d667880 into main Aug 2, 2023
@yaphott yaphott deleted the 0.0.4-release branch August 2, 2023 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants