Skip to content

Create AGENTS.md - #394

Merged
llrs-roche merged 12 commits into
mainfrom
391_CLAUDE
Oct 5, 2026
Merged

llrs-roche merged 12 commits into
mainfrom
391_CLAUDE

Conversation

@llrs-roche

Copy link
Copy Markdown
Member

Pull Request

Fixes #391

Note that on the url for the shared file I already replaced the name of the file.

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  -------------------
R/cdisc_data.R                   1       0  100.00%
R/deprecated.R                  18       0  100.00%
R/dummy_function.R               2       2  0.00%    15-16
R/formatters_var_labels.R       61       0  100.00%
R/join_key.R                    38       0  100.00%
R/join_keys-c.R                 12       0  100.00%
R/join_keys-extract.R          128       0  100.00%
R/join_keys-names.R             15       0  100.00%
R/join_keys-parents.R           30       0  100.00%
R/join_keys-print.R             45       0  100.00%
R/join_keys-utils.R             73       3  95.89%   35-38
R/join_keys.R                   21       0  100.00%
R/teal_data-class.R             36       6  83.33%   57-59, 66, 111-112
R/teal_data-constructor.R       11       2  81.82%   48, 51
R/teal_data-extract.R            3       0  100.00%
R/teal_data-get_code.R          13       8  38.46%   114-120, 124
R/teal_data-names.R              8       1  87.50%   42
R/teal_data-show.R               5       5  0.00%    14-20
R/topological_sort.R            32       0  100.00%
R/verify.R                      42      11  73.81%   67, 97-101, 104-108
TOTAL                          594      38  93.60%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: b95c9b2

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files   15 suites   1s ⏱️
163 tests 163 ✅ 0 💤 0 ❌
218 runs  218 ✅ 0 💤 0 ❌

Results for commit b95c9b2.

♻️ This comment has been updated with latest results.

@osenan osenan self-assigned this Sep 30, 2026

@osenan osenan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job! I think the PR is very ready, but it needs last fixes. First, to remove a line in the shared markdown only intended to avoid lint errors. Then there are comments related to the text form and organization, but the content is ok.

Comment thread vignettes/_AGENTS.Rmd Outdated
Comment thread R/teal_data-class.R Outdated
Comment thread vignettes/_AGENTS.Rmd Outdated
Comment thread vignettes/_AGENTS.Rmd Outdated
Comment thread AGENTS.md Outdated
Comment thread vignettes/_AGENTS.Rmd Outdated
Comment thread vignettes/_AGENTS.Rmd Outdated
llrs-roche and others added 6 commits September 30, 2026 15:06
Co-authored-by: Oriol Senan  <35930244+osenan@users.noreply.github.com>
Signed-off-by: Lluís Revilla <185338939+llrs-roche@users.noreply.github.com>
@llrs-roche
llrs-roche requested a review from osenan October 1, 2026 12:01

@osenan osenan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, only update the addition of the shared after changes on the shared template.

Comment thread vignettes/_AGENTS.Rmd
@llrs-roche
llrs-roche enabled auto-merge (squash) October 5, 2026 09:18
@llrs-roche
llrs-roche merged commit c272701 into main Oct 5, 2026
26 checks passed
@llrs-roche
llrs-roche deleted the 391_CLAUDE branch October 5, 2026 12:38
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add CLAUDE.md

2 participants