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

docs(news): tweak NEWS.md #1040

Merged
merged 3 commits into from
Apr 15, 2024
Merged

docs(news): tweak NEWS.md #1040

merged 3 commits into from
Apr 15, 2024

Conversation

eitsupi
Copy link
Collaborator

@eitsupi eitsupi commented Apr 15, 2024

No description provided.

@eitsupi

This comment was marked as resolved.

Copy link
Collaborator

@etiennebacher etiennebacher left a comment

Choose a reason for hiding this comment

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

Can you check whether this is fine on the rendered website?

@eitsupi
Copy link
Collaborator Author

eitsupi commented Apr 15, 2024

Can you check whether this is fine on the rendered website?

Yes, of course I checked locally.

image

As an aside, it would be useful to have an option to render only the NEWS.md file and skip the execution of the man file.
pkgdown have functions to render only certain files, I believe.

@eitsupi eitsupi merged commit f363382 into main Apr 15, 2024
2 checks passed
@eitsupi eitsupi deleted the tweak-news branch April 15, 2024 13:45
@etiennebacher
Copy link
Collaborator

For now we only have freeze = TRUE to skip files that didn't change

@eitsupi
Copy link
Collaborator Author

eitsupi commented Apr 15, 2024

For now we only have freeze = TRUE to skip files that didn't change

Yes, I have used it and seen the results. However, right after deleting the generated files, we need to wait until all man pages are rendered.

altdoc::render_docs(freeze = TRUE, parallel = TRUE, verbose = TRUE)

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