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: rework Plugins page into a tabular format #1889

Merged

Conversation

achieveordie
Copy link
Contributor

Fixes #1887

Summary:

  • Separate Antpath and PolyLineTextPath usage markdown files for better readability.
  • Convert the plugins page into a tabular format with hyperlinked names and the corresponding descriptions.

The old representation:
old

The new representation:
new

@achieveordie
Copy link
Contributor Author

CHANGES.txt is failing on the pre-commit check which is weird since that file has not been changed. Perhaps it is being carried on from a previous merge?

@achieveordie
Copy link
Contributor Author

Cannot replicate it locally either, but found a different file which wasn't touched before. Perhaps a difference between the local and CI pre-commit might be causing these?

image

@Conengmo
Copy link
Member

I see it the precommit fails on main as well, that's my fault, sorry! I edited that file in the web UI. I'll fix that soon, but you can ignore it here.

@Conengmo
Copy link
Member

This looks really good btw, I'll take a closer look asap but on first glance this should be easy to merge!

@Conengmo
Copy link
Member

I fixed it on main so if you update your branch, tests should pass. Again apologies!

@achieveordie
Copy link
Contributor Author

I fixed it on main so if you update your branch, tests should pass. Again apologies!

Done! :)

@Conengmo Conengmo changed the title Rework Plugins Page into a tabular format docs: rework Plugins page into a tabular format Mar 4, 2024
@Conengmo Conengmo merged commit a00c3c3 into python-visualization:main Mar 4, 2024
12 checks passed
@Conengmo
Copy link
Member

Conengmo commented Mar 4, 2024

Thanks!

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.

Improve the Plugins documentation by including a one-liner for each available plugin
2 participants