Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Switch API to IDE in getVersion() doc entry #304

Closed
wants to merge 1 commit into from
Closed

Switch API to IDE in getVersion() doc entry #304

wants to merge 1 commit into from

Conversation

coatless
Copy link

Switches a manpage title/description to emphasize this is reading RStudio.Version() information to describe the RStudio version being used and, not, the R package {rstudioapi}.

c.f. discussion in posit-dev/positron#4706 (comment)

…rmation to describe the RStudio version being used and, not, the R package rstudioapi.
@coatless
Copy link
Author

Any interest in merging this? Or, should I close out the PR as it's been sitting for ~2 months or so.

@kevinushey
Copy link
Contributor

Sorry, I must've missed this PR, but I think changes have since been made in line with this PR's approach.

@coatless
Copy link
Author

@kevinushey no worries, looks like you addressed this about a month (Oct 16th) back in: 3bd91a6

So, I'm going to close out this PR.

Trying to cut down on having stale repos/PRs.

@coatless coatless closed this Nov 13, 2024
@coatless coatless deleted the clarify-version-man-page branch November 13, 2024 00:11
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants