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

docs: remove outdated architecture section from readme #3291

Merged
merged 1 commit into from
Mar 19, 2024
Merged

docs: remove outdated architecture section from readme #3291

merged 1 commit into from
Mar 19, 2024

Conversation

mowies
Copy link
Member

@mowies mowies commented Mar 19, 2024 •
edited
Loading

This PR

@mowies mowies requested a review from a team as a code owner March 19, 2024 09:46
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Mar 19, 2024
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

StackScribe
StackScribe reviewed Mar 19, 2024
View reviewed changes
README.md Show resolved Hide resolved
StackScribe
StackScribe reviewed Mar 19, 2024
View reviewed changes
README.md Show resolved Hide resolved
StackScribe
StackScribe requested changes Mar 19, 2024
View reviewed changes
Copy link
Contributor

@StackScribe StackScribe left a comment

Choose a reason for hiding this comment

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

The deleted material needs to be incorporated into the components section before it is deleted and the information about how the mutating webhook should be turned into its own page in the components section

@mowies mowies merged commit 9ede4f4 into keptn:main Mar 19, 2024
37 checks passed
@mowies mowies deleted the architecture-diagram branch March 19, 2024 12:06
@keptn-bot keptn-bot mentioned this pull request Mar 19, 2024
Merged
Vickysomtee pushed a commit to Vickysomtee/keptn-lifecycle-toolkit that referenced this pull request Apr 23, 2024
@mowies @Vickysomtee
docs: remove outdated architecture section from readme (keptn#3291)
ce00a1f 
Signed-off-by: vickysomtee <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@odubajDT odubajDT odubajDT approved these changes

@StackScribe StackScribe StackScribe approved these changes

@bacherfl bacherfl bacherfl approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove remaining architecture info from README Update architecture overview diagram
4 participants
@mowies @bacherfl @StackScribe @odubajDT