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

Update #7

Merged
merged 31 commits into from
Jun 25, 2024
Merged

Update #7

merged 31 commits into from
Jun 25, 2024

Conversation

SebKrantz
Copy link
Owner

No description provided.

hadley and others added 30 commits June 6, 2024 12:47
* Correctly expand paths on Windows, and nudge towards fs paths

* Add comment to explain difference between fs path and base path
* Lint tests

* Update tests/testthat/test-package.R
* Make `document` defunct, and warn on `autolink_html()` usage

* Actually remove `document` as it was deprecated for 5 years.
* Combine section_init() and create_subdir()

* We can figure out `depth` based on whether or not a `subdir` is supplied
* Ensures all user facing functions have the same interface
* Brings together directory checking code in one place
* Check for files, not directory

Fixes #2644. Closes #2645
…cument `init_site()` + speed cli console output (#2642)

* use Posix line ending in pkgdown

* Use `withr::local_options(cli.num_colors = num_col, .local_envir = .frame)` in `section_init()`

* Document `init_site()` usage.

* Only call `init_site()` when lazy = FALSE, since `build_home()` will call it anyway if necessary.

* Add `local_options_console()` to `init_site()`

* Apply suggestions from code review

Co-authored-by: Hadley Wickham <[email protected]>

---------

Co-authored-by: Hadley Wickham <[email protected]>
To ensure that all pkgdown users get the latest verson
Co-authored-by: Hadley Wickham <[email protected]>
Includes tinytex to fix build errors
since `preview_path()` was deprecated
@SebKrantz SebKrantz merged commit b0850c7 into SebKrantz:main Jun 25, 2024
1 of 6 checks passed
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.