Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Store tag descriptions in inst/tags.yml #1402

Merged
merged 14 commits into from
Jul 15, 2022
Merged

Store tag descriptions in inst/tags.yml #1402

merged 14 commits into from
Jul 15, 2022

Conversation

hadley
Copy link
Member

@hadley hadley commented Jul 12, 2022

Fixes #1375. Fixes #1165.

@kevinushey
Copy link
Contributor

nit: maybe consider storing the tags in inst/roxygen/tags.yml? So that R packages could collect any roxygen-related metadata into that directory (and avoid a theoretical collision with something else at inst/tags.yml)

@hadley
Copy link
Member Author

hadley commented Jul 15, 2022

I could make it roxyen-tags.yml, but I'm not sure I want to imply a whole directly worth of installed/installable roxygen2 files.

@kevinushey
Copy link
Contributor

That makes sense to me.

@hadley hadley merged commit 4fd1e89 into main Jul 15, 2022
@hadley
Copy link
Member Author

hadley commented Jul 15, 2022

I'm going to merge this because I think it's a good start and I want to get a roxygen2 release out before conf. We can iterate more once y'all start working on for the IDE.

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.

Documentation for tags Maintain the complete list of roxygen tags as Rd topics
2 participants