Skip to content
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 update #664

Merged
merged 1 commit into from
Sep 21, 2023
Merged

Conversation

erikbosch
Copy link
Contributor

Fixes #659

Also clarifies jwt token naming for server and need to specify servername for tls

How to check links is described as part of release process at https://github.com/eclipse/kuksa.val/wiki/Release-Process#checking-links, I just improved it a bit.

Fixes eclipse#659

Also clarifies jwt token naming for server and
need to specify servername for tls
Copy link
Contributor

@lukasmittag lukasmittag 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 couldn't verify links by clicking but had a look if they make sense.

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.

Content-wise looks good to me.

I noticed, NOTICE.md might need to go away eventually, as it is not really current, and we are doing the Dashfiles in release and BOMs in the artifacts. But for no taking the editorial changes

This also means the logic in #661 is fine, right?

@SebastianSchildt SebastianSchildt merged commit 401e3c4 into eclipse:master Sep 21, 2023
9 checks passed
@erikbosch erikbosch deleted the erik_doc branch September 29, 2023 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Links broken
3 participants