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

Made plugin compatible with TypeDoc 0.22.x #447

Merged
merged 3 commits into from
Oct 24, 2021

Conversation

RunDevelopment
Copy link
Contributor

Hello @kamiazya!

Since I want this plugin in my docs, I updated everything to be compatible with TypeDoc 0.22.x. Unfortunately, I had to reimplement essentially the entire plugin, so almost nothing of the old code remains.

The new implementation only uses TypeDoc's public-facing API, so it will hopefully work with future TypeDoc versions for a long while.

I also had to remove E2E test since I couldn't get it to compile. The other tests have been updated.


One problem that I haven't addressed yet is that using an h4 in @mermaid tags looks strange.

image

Maybe we should use bold text instead? We could also display the h4 on the next line.


This fixes #441.

@kamiazya kamiazya self-requested a review October 22, 2021 11:06
Copy link
Owner

@kamiazya kamiazya left a comment

Choose a reason for hiding this comment

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

@RunDevelopment Nice work👍

Thanks!

@kamiazya kamiazya merged commit 09f8b4a into kamiazya:main Oct 24, 2021
@kamiazya
Copy link
Owner

@all-contributors please add @RunDevelopment for code, test

@allcontributors
Copy link
Contributor

@kamiazya

I've put up a pull request to add @RunDevelopment! 🎉

@RunDevelopment RunDevelopment deleted the issue441 branch October 24, 2021 11:32
@kamiazya kamiazya added the enhancement New feature or request label Oct 25, 2021
@azatoth
Copy link
Contributor

azatoth commented Oct 25, 2021

@RunDevelopment great work!

@kamiazya any chance of having this released in the near time?

@kamiazya
Copy link
Owner

@azatoth Yes.

The reason it couldn't be released is that the CI in the main branch is failing.

If this can be fixed, I'll release it, but I haven't had time.

(If you can fix it, please fix it🙏)

@azatoth
Copy link
Contributor

azatoth commented Oct 25, 2021

@azatoth Yes.

The reason it couldn't be released is that the CI in the main branch is failing.

If this can be fixed, I'll release it, but I haven't had time.

(If you can fix it, please fix itpray)

Oh, I didn't realize that; I've made a PR now at #452

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Breaking changes against typedoc 0.22.x
3 participants