-
-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
Remove usage of Directives in docs #4756
Comments
Hey @sidharthv96 can you provide a little more brief about this issue and some reference of what is to be done |
I can do this right after |
Ok I understand now so should I work on this now @sidharthv96 as @nirname mentioned he want to work on it but I can want to try contributing to this as I am new to open source and this issue will be good start |
@Aryangp no problem, I have plenty other things to do, if you are ready to commit - welcome |
I see that deprecation warnings are the part of #4750 so in this issue we have to tackle only examples of code, I think |
@nirname please see #4487 (comment) regarding assigning issues. |
I think it would be nice to have such script, it would help when implementing #4620 and give the ability to preform |
@Yokozuna59 see I ma new to this tell if I am wrong are you implying to something like a bash script to convert it into yml code |
@Aryangp A TS script would be better, so we could the logic in other places too. So a TS script that converts directives into yaml in |
ok i got that first i will make a ts script and try this approch and tell you the progress here |
sorry i was working on something else that why wasn't able to work on it , I will make a pr in a few days just want to give you update on progress of this issue and also i have made a script just like mentioned above i will share the code of the script on slack see if it need any improvement @sidharthv96 @Yokozuna59 |
@Aryangp Please, if you could, mention users at the beginning of your message so that we know who you're talking to :)
No worries.
Please create another PR for it. It would easier and better visibility for the community when there's an open PR. |
hey @sidharthv96 can you tell does the
is inportant in the final yaml code or is it just for formatting as just there is some space come in this --- after running the script so can you tell |
@Aryangp Hi! I am glad to hear that your are working on the task. As @Yokozuna59 had already mentioned, please open PR to share the source code, no matter how good or bad it is. It is much better to keep it as a history and share with the community. Sorry for answering the question you had addressed to another person:
We used to have different ways of configuring diagrams. Among the others was "directives" syntax, some JSON-like embedded in diagram, wrapped in Now we are going to switch to YAML instead of JSON and custom brackets We do not get rid of directives syntax, we are going to deprecate them. That being said:
```mermaid
---
title: This is another complicated flow
config:
theme: base
sankey:
nodeAlignment: justify
flowchart:
curve: cardinal
htmlLabels: false
---
// any diagram code goes here
``` It already works. Now we simply need to clean it up and replace all the remains of it in the docs. So the idea is either:
In case you are writing script for doing the job, please stick to the example provided. If the example is not as comprehensive as it could be, feel free to ask. |
hey @sidharthv96 and @Yokozuna59 i have made a pr where i have put the script can you check it out and provide any improvement |
and yeah @nirname thanks for explaining sorry i seen your message late |
Lets do this..... |
With #4750, the directives will be officially deprecated.
So all instances of their usage should be removed from the documentation.
The page for directives should remain, as a reference for people using older versions, but their usage from other examples in general should be replaced with the frontmatter.
The text was updated successfully, but these errors were encountered: