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

fix name of transformations #200

Merged
merged 1 commit into from
Jun 20, 2024
Merged

Conversation

maestro-1
Copy link
Contributor

In working recently, I noticed the names of some of the transformations were off.
Took a look and discovered that the f-string wasn't properly handled. This PR fixes that

@EntilZha
Copy link
Owner

EntilZha commented May 1, 2024

Thanks for the catch! I'm pretty sure the type hints PR will be ready to merge soon, so I'm going to try to get that merged first since merging this looks like a pretty trivial/easy rebase marge after that. Thanks!

@maestro-1
Copy link
Contributor Author

Okay, sounds good

Copy link

codecov bot commented May 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.12%. Comparing base (6ed2e9a) to head (1e26966).
Report is 5 commits behind head on master.

Current head 1e26966 differs from pull request most recent head 5a5b3f9

Please upload reports for the commit 5a5b3f9 to get more accurate results.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #200   +/-   ##
=======================================
  Coverage   98.12%   98.12%           
=======================================
  Files          12       12           
  Lines        2350     2350           
=======================================
  Hits         2306     2306           
  Misses         44       44           

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

@EntilZha EntilZha merged commit d1b6479 into EntilZha:master Jun 20, 2024
4 checks passed
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.

None yet

2 participants