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

docs: cleanup after theme migration #1045

Merged
merged 9 commits into from
Apr 5, 2023

Conversation

aepfli
Copy link
Member

@aepfli aepfli commented Mar 13, 2023

Within this pull request, we finalize the migration of our hugo theme tooling.

We extracted the theme into its own repository. And we cleaned it up from KLT-specific configuration. As an aftermath, we must set KLT-specific configurations within this repository. Additionally we centralized the docs release action also within the repository, and are now using the one from the docs-tooling repo

KLT-specifics are:

  • google site verification
  • github link in footer

closes: #1013

@netlify
Copy link

netlify bot commented Mar 13, 2023

Deploy Preview for keptn-lifecycle-toolkit ready!

Name Link
🔨 Latest commit 2171274
🔍 Latest deploy log https://app.netlify.com/sites/keptn-lifecycle-toolkit/deploys/642c036a6eac2b0008f57d0e
😎 Deploy Preview https://deploy-preview-1045--keptn-lifecycle-toolkit.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@aepfli aepfli force-pushed the docs/cleanup-after-theme-migration branch from 674cabe to 80b46e8 Compare March 14, 2023 08:26
@codecov
Copy link

codecov bot commented Mar 14, 2023

Codecov Report

Merging #1045 (0fdf584) into main (2c62593) will decrease coverage by 0.09%.
The diff coverage is n/a.

❗ Current head 0fdf584 differs from pull request most recent head 2171274. Consider uploading reports for the commit 2171274 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1045      +/-   ##
==========================================
- Coverage   59.15%   59.06%   -0.09%     
==========================================
  Files         128      128              
  Lines       10035    10035              
==========================================
- Hits         5936     5927       -9     
- Misses       3884     3890       +6     
- Partials      215      218       +3     

see 4 files with indirect coverage changes

Flag Coverage Δ
certificate-operator 64.15% <ø> (ø)
component-tests 59.97% <ø> (-0.87%) ⬇️
lifecycle-operator 78.48% <ø> (ø)
metrics-operator 77.57% <ø> (ø)
scheduler 21.75% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@aepfli aepfli force-pushed the docs/cleanup-after-theme-migration branch 3 times, most recently from 94d466e to 90cf1d8 Compare March 20, 2023 13:07
@aepfli aepfli force-pushed the docs/cleanup-after-theme-migration branch 2 times, most recently from 1e4f9c6 to 775a65f Compare March 20, 2023 13:20
@aepfli aepfli marked this pull request as ready for review March 29, 2023 09:05
@mowies
Copy link
Member

mowies commented Mar 29, 2023

@aepfli please provide a valid description where you explain what you did here :)

docs/README.md Outdated Show resolved Hide resolved
docs/static/google90258e79df368ea0.html Outdated Show resolved Hide resolved
mowies
mowies previously approved these changes Apr 4, 2023
aepfli and others added 9 commits April 4, 2023 12:58
Migrated theme, and now we can use a proper template

Signed-off-by: Simon Schrottner <[email protected]>
Signed-off-by: Simon Schrottner <[email protected]>
Signed-off-by: Simon Schrottner <[email protected]>
Signed-off-by: Simon Schrottner <[email protected]>
Signed-off-by: Simon Schrottner <[email protected]>
Co-authored-by: Moritz Wiesinger <[email protected]>
Signed-off-by: Simon Schrottner <[email protected]>
Signed-off-by: Simon Schrottner <[email protected]>
@aepfli aepfli force-pushed the docs/cleanup-after-theme-migration branch from 0fdf584 to 2171274 Compare April 4, 2023 11:00
@sonarcloud
Copy link

sonarcloud bot commented Apr 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@thisthat thisthat merged commit 0125462 into keptn:main Apr 5, 2023
@keptn-bot keptn-bot mentioned this pull request Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation ops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improvements regarding documentation theme
3 participants