Movatterモバイル変換


[0]ホーム

URL:


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

docs: remove mention of Chart Studio Cloud and enumeration of font names#7348

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
gvwilson merged 3 commits intomasterfromsimplifying-font-attribute-docs
Mar 18, 2025

Conversation

@gvwilson
Copy link
Contributor

Removing this text here means that it won't be duplicated multiple times in the plotly.py docs.

@gvwilsongvwilson self-assigned thisJan 27, 2025
@gvwilsongvwilson added P1needed for current cycle documentationwritten for humans labelsJan 27, 2025
@gvwilson
Copy link
ContributorAuthor

@LiamConnors if you have a minute, can you please review and either approve or suggest an alternative strategy?

Copy link
Member

@LiamConnorsLiamConnors left a comment

Choose a reason for hiding this comment

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

Looks good to me. Though, I think the schema needs to be regenerated for the tests to pass.

Removing this text here ensures it won't be present when plotly.py codeis generated, which reduces bundle size.
@gvwilsongvwilsonforce-pushed thesimplifying-font-attribute-docs branch fromc15cc34 to7640e3eCompareFebruary 25, 2025 15:56
@gvwilson
Copy link
ContributorAuthor

Re-created this PR on top ofmaster and regenerated schema - I believe the tests that are failing are also failing onmaster right now.

LiamConnors reacted with thumbs up emoji

@gvwilsongvwilson merged commitb1c9ce5 intomasterMar 18, 2025
1 check failed
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@LiamConnorsLiamConnorsLiamConnors approved these changes

Labels

documentationwritten for humansP1needed for current cycle

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gvwilson@LiamConnors

[8]ページ先頭

©2009-2025 Movatter.jp