Skip to content

dev UD - #85

Merged
vertesy merged 4 commits into
devfrom
main
Sep 2, 2026
Merged

vertesy merged 4 commits into
devfrom
main

Conversation

@vertesy

@vertesy vertesy commented Sep 2, 2026

Copy link
Copy Markdown
Owner

No description provided.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e1a7ee2517

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread AGENTS.md Outdated
- Do not use tests.
- Never update the package version unless the user explicitly requests a version change.
- Do not raise code review findings that ask for a package version change.
- In `/Development/MYPACKAGE/Development/Create_the_MYPACKAGE_Package.R")`, `PackageTools::document_and_create_package()` recreates an R package’s metadata and documentation from a configuration file. It runs `devtools::document()` to regenerate package documentation, including the DESCRIPTION and NAMESPACE, from roxygen annotation and `config.R`.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Point agents to the actual package-generation script

Problem: The documented path does not exist in this repository; the script is Development/Create_the_CodeAndRoll2_Package.R. Trigger: An agent following these instructions to regenerate package metadata will look for /Development/MYPACKAGE/Development/Create_the_MYPACKAGE_Package.R") and cannot run it. Fix: Replace the placeholder path with the real repository-relative path.

Useful? React with 👍 / 👎.

@vertesy

vertesy commented Sep 2, 2026

Copy link
Copy Markdown
Owner Author

@copilot please fix:

"Point agents to the actual package-generation script

Problem: The documented path does not exist in this repository; the script is Development/Create_the_CodeAndRoll2_Package.R. Trigger: An agent following these instructions to regenerate package metadata will look for /Development/MYPACKAGE/Development/Create_the_MYPACKAGE_Package.R") and cannot run it. Fix: Replace the placeholder path with the real repository-relative path."

Copilot AI commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

@vertesy I can’t apply these changes because the pull request’s branch is the repository’s default branch, which I’m not allowed to push to directly.

You can ask me to create a new pull request with the requested changes instead and I'll open a separate PR on a new branch.

Updated documentation references and paths for CodeAndRoll2 package.
@vertesy
vertesy merged commit 353bf68 into dev Sep 2, 2026
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