Skip to content

Clarify ECK license update doc #2766

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Sep 10, 2025
Merged

Clarify ECK license update doc #2766

merged 4 commits into from
Sep 10, 2025

Conversation

kunisen
Copy link
Contributor

@kunisen kunisen commented Sep 1, 2025

to address #2765

@kunisen kunisen self-assigned this Sep 1, 2025
@kunisen kunisen requested a review from a team as a code owner September 1, 2025 01:40
@kunisen kunisen added documentation Improvements or additions to documentation supportability ability enable self-service or support of product Team:Platform Issues owned by the Platform Docs Team labels Sep 1, 2025
@kunisen kunisen linked an issue Sep 1, 2025 that may be closed by this pull request
[Internal]: Clarify ECK license doc #2765
Closed
@kunisen kunisen changed the title Clarify ECK license update Clarify ECK license update doc Sep 1, 2025
@kunisen kunisen requested a review from pebrc September 1, 2025 01:41
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 1, 2025 •
edited
Loading

🔍 Preview links for changed docs

eedugon
eedugon approved these changes Sep 2, 2025
View reviewed changes
Copy link
Contributor

@eedugon eedugon left a comment

Choose a reason for hiding this comment

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

Looks great!

@kunisen @eedugon
Update deploy-manage/license/manage-your-license-in-eck.md
46f38df 
Co-authored-by: Edu González de la Herrán <25320357+eedugon@users.noreply.github.com>
barkbay
barkbay approved these changes Sep 8, 2025
View reviewed changes
Copy link
Contributor

@barkbay barkbay left a comment

Choose a reason for hiding this comment

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

LGTM

@kunisen kunisen enabled auto-merge (squash) September 10, 2025 01:03
@kunisen kunisen merged commit 15907bd into main Sep 10, 2025
6 of 7 checks passed
@kunisen kunisen deleted the kunisen-docpr-slack-20250901 branch September 10, 2025 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@barkbay barkbay barkbay approved these changes

@eedugon eedugon eedugon approved these changes

@pebrc pebrc Awaiting requested review from pebrc

Assignees

@kunisen kunisen

Labels

documentation Improvements or additions to documentation supportability ability enable self-service or support of product Team:Platform Issues owned by the Platform Docs Team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Internal]: Clarify ECK license doc

3 participants

@kunisen @barkbay @eedugon