Skip to content
This repository was archived by the owner on Dec 18, 2024. It is now read-only.

*: Rename kuksa-viss-client kuksa-client #379

Conversation

romainletendart
Copy link
Contributor

Package directory and name shall use a dash as per https://pep8.org/#package-and-module-names.
Entry point shall also use a dash as it is common for UNIX programs. The imported python package has to use an underscore however otherwise we cannot import it (SyntaxError).

Fixes #318.

@romainletendart romainletendart force-pushed the romainletendart/rename-kuksa-client branch 2 times, most recently from 7c78a3a to 7a6d78c Compare October 12, 2022 13:21
:oo. oho -+: -hh/
.+o+-`oho `:shy-
./o/ohy//+oyhho-
`-/+oo+/:.

Default tokens directory: /home/erik/.local/lib/python3.9/site-packages/kuksa_certificates/jwt
Copy link
Contributor

Choose a reason for hiding this comment

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

Not an issue of this PR, but we might want to change this later

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would keep that for later then.

@@ -1,22 +1,22 @@
{"version": 2, "width": 80, "height": 33, "ts": 1599119755, "env": {"SHELL": "/usr/bin/fish", "TERM": "rxvt-unicode-256color"}}
[0.028694, "o", "\u001b]0;fish /home/wenwen/tools/kuksa.val"]
Copy link
Contributor

Choose a reason for hiding this comment

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

If we change these can we at least also re-render them? See https://github.com/eclipse/kuksa.val/blob/master/doc/pictures/README.md

(shall we also redo them (e.g. record fresh .casts) maybe, but if not super easy maybe not in this PR)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I re-rendered the GIFs.
Let's keep re-recording for later I would say.

Copy link
Contributor

@SebastianSchildt SebastianSchildt left a comment

Choose a reason for hiding this comment

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

I tired the built docker and it still worked (the VISS part at least). Changes look good, I do think at the very least we should generate updated gifs from the modified .cast fiels (see comment above). Otherwise I'd rather merge this fast and "fail forward", because this PR touches a lot of files, and keeping it open too long will make things painful...

Copy link
Contributor

@erikbosch erikbosch left a comment

Choose a reason for hiding this comment

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

Looks good to me if what Sebastian points out is fixed.

(Maybe we should create some release instruction somewhere where it is mentioned that it shall be reviewed if the examples are still sufficiently relevant, or if they need to be regenerated)

Package directory and name shall use a dash as per
https://pep8.org/#package-and-module-names.
Entry point shall also use a dash as it is common for UNIX programs.
The imported python package has to use an underscore however otherwise
we cannot import it (SyntaxError).

Fixes eclipse-archived#318.

Signed-off-by: Romain Létendart <romain.letendart@fr.bosch.com>
@romainletendart romainletendart force-pushed the romainletendart/rename-kuksa-client branch from 7a6d78c to 11cb0cd Compare October 13, 2022 13:17
Copy link
Contributor

@SebastianSchildt SebastianSchildt left a comment

Choose a reason for hiding this comment

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

Looking good, lgtm 🦤

@SebastianSchildt SebastianSchildt merged commit acd0110 into eclipse-archived:master Oct 13, 2022
@romainletendart romainletendart deleted the romainletendart/rename-kuksa-client branch November 9, 2022 13:25
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename kuksa-viss-client to kuksa-client
3 participants