Skip to content

Updates to R wrapping. - #2605

Merged
zivy merged 1 commit into
SimpleITK:mainfrom
zivy:updatesToRWrapping
May 22, 2026
Merged

zivy merged 1 commit into
SimpleITK:mainfrom
zivy:updatesToRWrapping

Conversation

@zivy

@zivy zivy commented May 22, 2026

Copy link
Copy Markdown
Member

Updating URLs and contents of DESCRIPTION file
which is part of the R package.

email = "[email protected]"))
Depends: R (>= 3.3)
email = "[email protected]"))
Author: Richard Beare, Bradley Lowekamp, Ziv Yaniv plus loads of others

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

May want to say a more formal and short variation of "The Insight Software Consortium and the ITK user and developer communities."

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done

Comment thread Wrapping/R/sitkRArray.cxx Outdated
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0.txt
* https://www.apache.org/licenses/LICENSE-2.0.txt

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

None of the links to apache.org use https. Please do not diverse these few files.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

undone.

Updating URLs and contents of DESCRIPTION file
which is part of the R package.
@zivy
zivy force-pushed the updatesToRWrapping branch from b2aef31 to 7999262 Compare May 22, 2026 14:19
@blowekamp
blowekamp self-requested a review May 22, 2026 17:11

@blowekamp blowekamp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@zivy
zivy merged commit 13bf8bb into SimpleITK:main May 22, 2026
10 checks passed
@zivy
zivy deleted the updatesToRWrapping branch May 22, 2026 17:31
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