workshop icon indicating copy to clipboard operation
workshop copied to clipboard

Remove Swarm references from Japanese translation

Open alexellis opened this issue 3 years ago • 11 comments

Expected Behaviour

Remove remaining Docker Swarm references

Current Behaviour

Swarm isn't supported by OpenFaaS, but it is mentioned here in some files.

Possible Solution

Work through each file and update it, also remove wording such as "If on Kubernetes..."

The workshop won't have support for faasd, which has a separate eBook and guide.

alexellis avatar Mar 11 '21 22:03 alexellis

Japanese translations would probably have to be updated too.

kadern0 avatar Mar 16 '21 21:03 kadern0

I'm not sure if we are in contact with any Japanese speakers. Could you look at the git commit history and ping a couple of people here?

alexellis avatar Mar 17 '21 07:03 alexellis

Hi @kenfdev and @toshi0607, I've removed all the Docker Swarm references from the labs, could you do the same for the Japanese translation?

kadern0 avatar Mar 20 '21 23:03 kadern0

/add label: help wanted, good first issue

alexellis avatar Nov 24 '21 16:11 alexellis

/set title: Remove Swarm references from Japanese translation

alexellis avatar Nov 24 '21 16:11 alexellis

@alexellis Hi, not sure the progress here but does the issue scope only refer to removal of Docker Swarm related lines which https://github.com/openfaas/workshop/pull/193 covers? If so I wouldn't mind doing it but not sure if there's anything else that has to be updated.

inductor avatar Nov 29 '21 09:11 inductor

It's the equivalent changes, but in the Japanese README file. Does that make sense?

https://github.com/openfaas/workshop/tree/master/translations/ja

alexellis avatar Nov 29 '21 09:11 alexellis

Is only README fine? or do you want to change all of the labs?

inductor avatar Nov 29 '21 09:11 inductor

Anywhere you see the word "Swarm", remove those lines. If the surrounding lines need updating we will need that too, for instance:

"Pick one of the two options of Swarm and Kubernetes" <- that sentence needs reformatting.

But if it says: "Now run kubectl get pods. On Swarm run..." You can just remove the Swarm part.

So yes, check each of the files.

alexellis avatar Nov 29 '21 09:11 alexellis

@alexellis I'm having a look at differences but the Japanese translation looks very outdated.

For example the setup a single-node cluster section in lab1 in English has the following diff.

- The labs use Kubernetes, and still have some references to Docker Swarm. Please ignore those references as Swarm is no longer supported by the OpenFaaS community. Some of the workshop labs will work for faasd, but you may need to alter the commands and we do not provide support for the workshop when taken with faasd.
+ The labs use Kubernetes, Swarm is no longer supported by the OpenFaaS community. Some of the workshop labs will work for faasd, but you may need to alter the commands and we do not provide support for the workshop when taken with faasd.

https://github.com/openfaas/workshop/pull/193/files#diff-097f79b5e813d46f4c17d6e5a2635bb4c068854e07cbfb5e63dd037242e3d8aeL93

But Japanese doesn't have this whole sentence and it includes a dedicated section for Docker Swarm that implies something like "You can use either Kubernetes and Docker Swarm but in a workshop you probably prefer to use Swarm rather than Kubernetes because it's easier to setup.". So I would say reconstruction would be required.

https://github.com/openfaas/workshop/blame/master/translations/ja/lab1.md#L41

I would not mind just removing Swarm related lines but it won't be up to date and updating the whole translation should be done in another PR.

Any opinions?

inductor avatar Nov 29 '21 11:11 inductor

For now, I think it would be helpful to prevent new users from assuming Swarm is valid and trying that out.

Synchronising the translation is an optional technical debt item in my eyes.

alexellis avatar Nov 29 '21 15:11 alexellis