-
Notifications
You must be signed in to change notification settings - Fork 134
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
Documentation: Refactoring of plugins.md #690
base: main
Are you sure you want to change the base?
Conversation
In dem PR verwende ich ds one sentence per line pattern, was es bei PR request reviews (und bei aenderungs vorschlaegen) deutlich einfacher macht. Für das Rendern macht es keinen Unterschied. |
Das ist gut. Das machen wir auch so. |
@naltatis gibt es eine Moeglichkeit, labels in der Dokumentation darzustellen ? Ich wuerde gerne zwei labels fur |
Ich bin den aktuellen Stand gestern einmal durchgegangen und hab dabei ein paar sprachliche Dinge mit angepasst. Die Struktur und der Erklärungslevel gefällt mir total gut. Willst du das Umhängen Zum "Plugin" Namen: Ich verstehe die Argumentation. Das ist nur aktuell schon an so vielen Stellen in Code und Sprachgebrauch, dass das schwer zu ändern ist ohne unnötige Verwirrung zu erzeugen. |
@rhuss Für die Darstellung könntest du
Optisch finde ich den |
@naltatis ich hab mal meine verstaubten CSS Kenntnisse ausgegraben und eine simple React Komponente |
Ich habe ein die Beispiele zu Als naechstes werde ich versuchen, die Tabellen sinnvoll zu ergaenzen. Da braeuchte ich spaeter dann ebenfalls die Hilfe von euch SMEs :) |
Das wuerde ich dann am Ende machen wenn wir den Inhalt soweit festgezurrt haben. |
|
||
Das Schema hat dabei immer folgende Struktur: | ||
|
||
```yaml {3,5-6,8} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@naltatis Hast Du eine Idee, wie ich die Platzhalter (<...>
) hervorheben kann ? Ich habs mit Prism plugins probiert, aber React/Docsaurus-Fu ist nicht gut genug. Idealerweise waeren die Platzhale in kursiv + bold.
Ist aber nicht dringend, man kann sicher auch ohne leben. Nur falls Du zufaellig was weisst.
Testweise habe ich line-highlight Prism plugin enabled, aber so toll ist das hier auch nicht (sollte man wohl auch mit magic comments
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line-Highlights mit Magic Comments sollten ja ohne Konfigurationsänderung schön möglich sein:
tariffs:
currency: EUR # (default EUR)
grid:
// highlight-start
type: fixed
// highlight-end
price: 0.294 # EUR/kWh
Aber du möchtest nur die Teilwörter, keine Zeilen richtig? Da bin ich auch überfragt.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ja, genau. Ich wollte die Platzhalter attr1
bzw attr2
hervorheben.
Aber kein Problem, das ist nur Nebensache. Inhalt ist wichtiger ;-)
Danke fuers Feedback.
Hier mein erster Wurf für eine Restrukturierung von
plugins.md
. Dabei liegt der Fokus auf der Kapitelstruktur, es sind noch einige Gaps offen:Out-of-scope in diesem PR:
Beim Review koennt ihr gerne direkt korrigieren und auf den Branch pushen, er ist offen für Maintainers (vielleicht bei groesseren Aenderung noch einen Review Kommentar).
Random Opinion following ..., nicht wichtig :-)
Darueberhinaus habe ich noch ein paar kleine Anmerkungen (sorry, ich kann nicht anders :-), die aber für den PR nicht relevant sind:"Plugin" ist eine etwas unglueckliche Bezeichnung, das sie in vielen anderen Faellen, externen code und Konventionen bezeichnen, die ausserhalb des Kernprojekts leben und vorgegebene APIs implementieren. Im Falle von evcc ist aber die Liste der Plugins fix und koennen nicht selbst ergaenzt werden. Z.b. muesste man für eine Resol VBus Integrationen den core-code erweitern. In meinen Augen handelt es sich vielmehr um Integrationen, bei der externe Sources (oder Producer) und Sinks (oder Consumer) eingebunden werden, so wie sie z.b. in dem klassischen EAI Patterns Buch beschrieben sind. Mir ist klar das im Kontext von evcc "Integrationen" anders interpretiert wird (i.e. wie man evcc in andere Systeme integrieren kann, also mehr eine "API" Beschreibung), daher sollten man es auch so lassen. Ich will hier keine Diskussion starten, sondern mein innerer Spiesser wollte das einfach loswerden :-P und evtl. hilft das in Diskussionen um Missverstaendnisse zu vermeiden.
Das "combined status" plugin passt in meinen Augen nicht wirklich in die Liste, da es nicht für alle Geraete genutzt werden kann (für
meter
macht das keinen Sinn, oder ?). Vielmehr wuerde ich es als Eigenschaft vonvehicle
sehen, bei denen Plugins genutzt werden koenne. Your mileage may vary.