Skip to content

Fixed a bunch of small Ropenbt R package issues. - #56

Merged
jared321 merged 4 commits into
mainfrom
mp-Ropenbt-fixes
Jul 28, 2026
Merged

Fixed a bunch of small Ropenbt R package issues.#56
jared321 merged 4 commits into
mainfrom
mp-Ropenbt-fixes

Conversation

@mpratola

Copy link
Copy Markdown
Collaborator

No description provided.

@mpratola
mpratola requested a review from jared321 July 17, 2026 06:11
Synchronize Feature Branch

@jared321 jared321 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.

If you change Ropenbt/LICENSE from a file to a symlink to the LICENSE file in the root of the repo, will the R packaging system include the license file correctly?

This is what I do for the Python package so that we (hopefully) only have to manage one license file.

@mpratola

Copy link
Copy Markdown
Collaborator Author

I would guess so, can try it. I should do ln -s ?

@jared321

Copy link
Copy Markdown
Contributor

That is what I have done in the past.

@mpratola

Copy link
Copy Markdown
Collaborator Author

Ok I fixed this and pushed.

@jared321 jared321 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.

  • Reviewed all changes
  • Confirm all actions passing

The latter was just a formality since we don't have any R actions yet.

@jared321
jared321 merged commit 0d7a97c into main Jul 28, 2026
83 checks passed
@jared321 jared321 mentioned this pull request Jul 28, 2026
@jared321
jared321 deleted the mp-Ropenbt-fixes branch July 28, 2026 20:33
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