Skip to content

Update old field classification naming#107

Merged
jatkinson1000 merged 1 commit into
mainfrom
old_agb_naming_consistency
Dec 15, 2024
Merged

Update old field classification naming#107
jatkinson1000 merged 1 commit into
mainfrom
old_agb_naming_consistency

Conversation

@jatkinson1000

Copy link
Copy Markdown
Owner

To be consistent with old indoor classification naming.

@TomHall2020

Copy link
Copy Markdown
Contributor

Yup, I definitely would have wanted to make that rename as part of the structure of #103 but good to have it locked in place already and I wouldn't have caught renaming the test functions and extra bits of documentation.

@jatkinson1000 jatkinson1000 force-pushed the old_agb_naming_consistency branch from 4f5fb06 to 70cd955 Compare December 15, 2024 15:30
@jatkinson1000 jatkinson1000 merged commit 470d14e into main Dec 15, 2024
@jatkinson1000 jatkinson1000 deleted the old_agb_naming_consistency branch December 15, 2024 15:38
@codecov

codecov Bot commented Mar 15, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.81%. Comparing base (e0d8d00) to head (70cd955).
Report is 19 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #107   +/-   ##
=======================================
  Coverage   97.81%   97.81%           
=======================================
  Files          30       30           
  Lines        1742     1742           
=======================================
  Hits         1704     1704           
  Misses         38       38           
Files with missing lines Coverage Δ
archeryutils/classifications/__init__.py 100.00% <ø> (ø)
...s/classifications/agb_old_field_classifications.py 100.00% <100.00%> (ø)
...cheryutils/classifications/tests/test_agb_field.py 100.00% <ø> (ø)
...yutils/classifications/tests/test_agb_old_field.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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