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.

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

Support multiple inheritance for doc generation #406

Merged
merged 41 commits into from
Jan 11, 2024
Merged

Support multiple inheritance for doc generation #406

merged 41 commits into from
Jan 11, 2024

Conversation

sujuka99
Copy link
Contributor

@sujuka99 sujuka99 commented Jan 10, 2024 •
edited
Loading

Preceeds #398

disrupted and others added 29 commits January 2, 2024 12:29
@disrupted
Refactor workaround for setting Helm app nameOverride
62bfb03
@disrupted
Update snapshot assertions
55089be
@disrupted
Use consistent naming for app values
8d9bb60
@disrupted
Rename fixture
934e4ed
@disrupted
Refactor
1c4e313
@disrupted
Rename
2309a7b
@disrupted
Merge remote-tracking branch 'origin/v3' into refactor/helm-nameoverride
7c8ff89
@disrupted
Refactor streams-bootstrap cleanup
f67a23f
@disrupted
Update tests
0578cb0
@disrupted
Inherit from common streams-bootstrap app
44195e7
@disrupted
Remove desc
a2a8418
@disrupted
Fix failing hooks
3457d1f
@disrupted
Merge remote-tracking branch 'origin/v3' into refactor/streams-bootst…
6ac39da 
…rap-cleanup
@disrupted
Cosmetic
a725f03
@disrupted
Cosmetic
26ac7f9
@disrupted
Rename run to clean
b193c03
@disrupted
Fix returned object type
9d27adc
@disrupted
Add pydocs and todo
c517a3b
@disrupted
Create streams-bootstrap base for all components based on its Helm
11dcc1e 
charts
@disrupted
Cleanup KPOps components import
d9786dd
@disrupted
Fix inheritance order of streams-boostrap apps
a49b551
@disrupted
Update components hierarchy diagram
2e2c7ba
@disrupted
Fix docs
02c0ef7
@disrupted
Merge branch 'v3' into refactor/streams-bootstrap-cleanup
70cf1f3
@disrupted
Update defaults schema
83ac398
@disrupted
Add docs for streams-bootstrap
45c58d8
@disrupted
Add docs for streams-bootstrap
7c92421
@sujuka99
refactor: extract function to utils
18cece3
@sujuka99
refactor: enable multiple inheritance for doc gen
d9808f8
@sujuka99 sujuka99 added the area/documentation Improvements or additions to documentation label Jan 10, 2024
@sujuka99 sujuka99 added type/refactor Refactoring of existing functionality area/CI Relates to GitHub actions labels Jan 10, 2024
@sujuka99 sujuka99 added this to the v3.0 milestone Jan 10, 2024
@sujuka99 sujuka99 self-assigned this Jan 10, 2024
@sujuka99 sujuka99 changed the base branch from refactor/streams-bootstrap-cleanup to v3 January 11, 2024 14:09
@disrupted disrupted mentioned this pull request Jan 11, 2024
Merged
3 tasks
@sujuka99 sujuka99 merged commit 6bd6e7c into v3 Jan 11, 2024
6 of 7 checks passed
@sujuka99 sujuka99 deleted the ci/support-multiple-inheritance branch January 11, 2024 16:14
disrupted added a commit that referenced this pull request Jan 15, 2024
@disrupted
Refactor streams-bootstrap cleanup jobs as individual HelmApp (#398)
bd9e135 
Depends on #397 

extracted the cleanup job needed for streams-bootstrap producer and
streams apps to its own HelmApp nested inside the parent component

- [x] Create docs for `StreamsBootstrap` base
- [x] Update KPOps component inheritance graph
- [x] Update docs gen for multiple component inheritance (#406)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@disrupted disrupted disrupted approved these changes

@raminqaf raminqaf Awaiting requested review from raminqaf

@irux irux Awaiting requested review from irux

Assignees

@sujuka99 sujuka99

Labels
area/CI Relates to GitHub actions area/documentation Improvements or additions to documentation type/refactor Refactoring of existing functionality
Projects
None yet
Milestone
v3.0
Development

Successfully merging this pull request may close these issues.
2 participants
@sujuka99 @disrupted