[DO NOT MERGE] Documentation update review

canonical / kafka-operator

Kafka VM operator

Apache License 2.0

6 stars 12 forks source link

[DO NOT MERGE] Documentation update review #172

Closed deusebio closed 4 months ago

deusebio commented 4 months ago

@grobbie @marcoppenheimer @zmraul @a-velasco

deusebio commented 4 months ago

I'm not sure there is any plan in Juju to also have remove-integration. I would agree that it would make things more consistent, but if there is no actual plan, we should not constraint one to another. Either way, I suppose we should move away from juju relate as this appears with deprecated warnings in the logs.

Anyhow, I don't mind too much. Up to @a-velasco to make a call such that we keep documentation consistent across DP.

deusebio commented 4 months ago

Actually I just found this just spot on.

TLDR remove-relation still makes sense.

integrate is the verb, relation is the noun. So the action/verb is integrating, but whenever we refer to something, it is a "relation", e.g. "Integrating is the action of creating a relation`

deusebio commented 4 months ago

superseded by #175