Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

CompatHelper: bump compat for Documenter to 1 for package docs, (keep existing compat) #525

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

Open
github-actions wants to merge 5 commits into master
base: master
Choose a base branch
Loading
from compathelper/new_version/2023-09-16-00-52-50-289-02299168519

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Sep 16, 2023

This pull request changes the compat entry for the Documenter package from 0.27 to 0.27, 1 for package docs.
This keeps the compat entries for earlier versions.

Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.

@theogf theogf force-pushed the compathelper/new_version/2023-09-16-00-52-50-289-02299168519 branch from fba631a to e047f40 Compare September 16, 2023 00:52
Copy link

codecov bot commented Sep 16, 2023
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.88%. Comparing base (2a6e5e9) to head (90e85b5).

Additional details and impacted files
@@ Coverage Diff @@
## master #525 +/- ##
===========================================
+ Coverage 0.42% 90.88% +90.46% 
===========================================
 Files 52 52 
 Lines 1423 1459 +36 
===========================================
+ Hits 6 1326 +1320 
+ Misses 1417 133 -1284 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

theogf commented Sep 24, 2023

It looks like with the new HTML size threshold the docs cannot be built.

Copy link
Member

It looks like with the new HTML size threshold the docs cannot be built.

Oooo fun. I wonder if this suggests that we should lower the resolution of some of our figures? My understanding of the size threshold is that it's intended to suggest to you that your page is really quite large, and is probably going to cause issues for users. Do you know if I've got that right?

Copy link
Member

theogf commented Sep 25, 2023

It looks like with the new HTML size threshold the docs cannot be built.

Oooo fun. I wonder if this suggests that we should lower the resolution of some of our figures? My understanding of the size threshold is that it's intended to suggest to you that your page is really quite large, and is probably going to cause issues for users. Do you know if I've got that right?

Yes that's the idea. Of course there's also the solution to raise the threshold...

Copy link
Member

Ahhh yeah, that might be better. I mean, no one has complained thus far

@simsurace simsurace deleted the compathelper/new_version/2023-09-16-00-52-50-289-02299168519 branch November 4, 2023 16:47
@simsurace simsurace restored the compathelper/new_version/2023-09-16-00-52-50-289-02299168519 branch November 4, 2023 16:48
Copy link
Member

Sorry, I deleted the branch by accident.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devmotion devmotion devmotion left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /