Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix: Change format step to check mode and comment suggest #701

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.

Sign up for GitHub

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

Jump to bottom
Merged
cristinamullin merged 3 commits into from
Oct 20, 2025
Merged

Fix: Change format step to check mode and comment suggest #701

cristinamullin merged 3 commits into from
Oct 20, 2025

Conversation

@jbousquin
Copy link
Collaborator

@jbousquin jbousquin commented Oct 17, 2025

Updated format step to check formatting instead of applying it. Commented out suggest step for reviewdog since it has setup not on approved list

@jbousquin
Change format step to check mode and comment suggest
1de0970 
Updated format step to check formatting instead of applying it. Commented out suggest step for reviewdog since it has setup not on approved list
@jbousquin
Merge branch 'develop' into 674-air
dfadd15
@github-actions
Copy link
Contributor

github-actions bot commented Oct 17, 2025 •
edited
Loading

coverage-report

File Coverage Missing
All files 33% ✅
R/ATTAINSCrosswalks.R 22% ✅ 64-742 931-941 945-949 954-957 962 970-973 980-983 988-1009 1017-1027 1052-1055 1179-1254 1266 1273-1451 1621-1633 1637-1640 1645-1647 1675-1678 1689-1692 1700-1704 1716-1720 1732-1738 1743-1754 1760 1768-1771 1861-1867 1872-1874 1979 1986-2157 2338-2724 2850-2853 2868 2876-2879 2886-2904 2912-2916 2927-2931 2947-3009 3056-3146 3151-3154 3159-3253
R/ATTAINSRefTables.R 0% ❌ 17-226
R/autoClean.R 85% ✅ 145-146 230-236 364-365 375-379
R/autoFilter.R 0% ❌ 27-430
R/CensoredDataSuite.R 88% ✅ 52-53 143 173-174 216-218 304-305 408-409 414 417 425 453-457 460-461 467-469 486-488
R/ContinuousDataFunctions.R 53% ✅ 151-152 184 189-259 266-267 290-328 345-349 352 392-393 483-547 552-596 603-610 615-616 655 666 672 677-684 709
R/CriteriaComparison.R 89% ✅ 159-161 166 176 223-241 383
R/CriteriaMethods.R 23% ✅ 155 175 185 191-250 404-406 422-424 529-669 675-711 733-984 1004-1172
R/CriteriaRefTables.R 0% ❌ 26-83
R/DataDiscoveryRetrieval.R 33% ✅ 197 206-211 225-230 240-245 251 255 270-676 690 698 700 704-707 710 716 718 722 724 730 734 736 746 748 752 758 760 764 766 771 773 777 783-786 789 801-809 822-828 857-865 879-887 907 1001-1033 1138-1145 1236-1272 1360-1363 1415-1574
R/DepthProfile.R 0% ❌ 96-1471
R/Figures.R 0% ❌ 64-1520
R/GeospatialFunctions.R 18% ✅ 70 163-165 176-180 184 239 277 321-548 689-690 696-699 708-922 927-1111 1279-1290 1307-1309 1311 1339 1345-1389 1416-1418 1496-1579 1589-2123 2203 2246 2267-3738
R/MaintenanceScheduled.R 0% ❌ 42-366
R/RequiredCols.R 16% ✅ 359-581
R/ResultFlagsDependent.R 63% ✅ 57 62 92-96 116-120 216 242-244 261-263 270 286-297 393 400 407 415 472 478 494-503 569 581-588 610 621-625 693-780 876 925-933 954 958 964-968
R/ResultFlagsIndependent.R 48% ✅ 69 75 108-134 238 246 253 260 265 276 348-349 366-403 491 496 503 579-583 594-609 697 702 708 785-963 1009 1028-1047 1058-1061 1163 1167 1210-1221 1226 1230-1233 1299-1537
R/Tables.R 77% ✅ 19-30 86
R/TADAGeospatialRefLayers.R 0% ❌ 8-13
R/TADARefTables.R 83% ✅ 72 82-86
R/Transformations.R 88% ✅ 76-77 82 179-185 343-344 380-381 465 540-541 709-722 808-809 820-821 826 841-844 847-856 860
R/UnitConversions.R 81% ✅ 128 344 351 358 365 372 379 386 393-394 520-584 608-645 668 708 722-731 923-925 1002 1059-1062 1092
R/Utilities.R 50% ✅ 28-32 183 299-300 304 309 382 484-492 553-554 561 623-624 654-658 758-759 769-773 778-779 822-1115 1141-1142 1153-1154 1214-1563 1656-1689
R/WQPWQXRefTables.R 63% ✅ 21-78 102 112 118-120 132 154 164 170-172 247 269 279 285-287 371 392 402 408-412 556 578 588 594-596 613 637 647 653-655 760-963

Minimum allowed coverage is 10%

Generated by 🐒 cobertura-action against 1de0970

@cristinamullin cristinamullin merged commit 9e1efae into develop Oct 20, 2025
7 of 9 checks passed
@cristinamullin cristinamullin deleted the 674-air branch October 20, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cristinamullin cristinamullin cristinamullin approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jbousquin @cristinamullin