Skip to content

0.5.0 checked refactor#84

Open
maksymiuks wants to merge 61 commits intomainfrom
dev
Open

0.5.0 checked refactor#84
maksymiuks wants to merge 61 commits intomainfrom
dev

Conversation

@maksymiuks
Copy link
Collaborator

Complete overhaul of the package. Thanks @dgkf for an excellent job on it! We need to merge it today, else checked will be taken down from CRAN

dgkf-roche and others added 15 commits June 16, 2025 12:10
Improve the readability of node names
Remove relation edge attribute and reporter edges
* Decycle graph

* Decycle the graph

* Make sure paths are properly dispatched

* Polish typos

* Add comments and fix lintr

* Add comments and fix lintr

* chore: refactor task_graph_sort (#65)

* Implement source dependencies management

* Revert sorting

---------

Co-authored-by: dgkf-roche <91694157+dgkf-roche@users.noreply.github.com>
* Initial remotes support

* Reintroduce options package

* 68 adjust results generation to new meta tasks structure (#71)

* Refactor results generation

* Add local plans (#70)

* Add local plans

* Remove line added by accident

* Make lintr happy

* Enhance source path check

* Polish results generation. Remove results_to_df

* Deduplicate graph

* Add edges types to remotes packages

* Fix lintr

* Make check pass

* Update rd names
* Initial basic tty suport

* Fix ansi and basic reporters
* Add ansi tty errors

* Rework ansi reporter

* Make lintr happy

* Change version

* Update R/checker.R

Co-authored-by: Doug Kelkhoff <18220321+dgkf@users.noreply.github.com>

* gc when needed

* Apply suggestions

---------

Co-authored-by: Doug Kelkhoff <18220321+dgkf@users.noreply.github.com>
* Reintroduce tests

* Refactor rcmd check process output

* Fix accidental too big incrementation

* Update tests

* fix lintr
* Reintroduce tests

* Refactor rcmd check process output

* Update documentation and readme

* Fix accidental too big incrementation

* Update readme and documentation
* Upgrade package version

* Make lintr happy
@maksymiuks maksymiuks mentioned this pull request Feb 18, 2026
7 tasks
@codecov
Copy link

codecov bot commented Feb 18, 2026

Codecov Report

❌ Patch coverage is 77.67978% with 329 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.34%. Comparing base (3615af8) to head (335f4be).

Files with missing lines Patch % Lines
R/task_graph.R 52.10% 91 Missing ⚠️
R/remotes.R 40.29% 40 Missing ⚠️
R/pkg_origin.R 68.80% 34 Missing ⚠️
R/results.R 69.07% 30 Missing ⚠️
R/reporter_ansi_tty.R 89.65% 27 Missing ⚠️
R/plan.R 85.27% 19 Missing ⚠️
R/utils-cli.R 76.38% 17 Missing ⚠️
R/task-formatting.R 31.81% 15 Missing ⚠️
R/utils-paths.R 64.70% 12 Missing ⚠️
R/run.R 53.33% 7 Missing ⚠️
... and 14 more
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #84       +/-   ##
===========================================
- Coverage   90.27%   79.34%   -10.94%     
===========================================
  Files          21       29        +8     
  Lines        1471     2159      +688     
===========================================
+ Hits         1328     1713      +385     
- Misses        143      446      +303     

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

🚀 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

Comments