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: Add /healthz/readiness endpoint #2455

Merged
merged 4 commits into from
Sep 10, 2024
Merged

docs: Add /healthz/readiness endpoint #2455

merged 4 commits into from
Sep 10, 2024

Conversation

ivov
Copy link
Contributor

@ivov ivov commented Sep 5, 2024

Copy link

netlify bot commented Sep 5, 2024

Deploy Preview for n8n-docs ready!

Name Link
🔨 Latest commit 9e0b88a
🔍 Latest deploy log https://app.netlify.com/sites/n8n-docs/deploys/66d9b813d559c60008e7c95b
😎 Deploy Preview https://deploy-preview-2455--n8n-docs.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 configuration.

Copy link

github-actions bot commented Sep 5, 2024

Overall readability score: 43.26 (🟢 +0.01)

File Readability
monitoring.md 61.68 (🟢 +6.37)
View detailed metrics

🟢 - Shows an increase in readability
🔴 - Shows a decrease in readability

File Readability FRE GF ARI CLI DCRS
monitoring.md 61.68 50.33 9.35 10.2 11.82 8.03
  🟢 +6.37 🔴 -0.4 🟢 +0.03 🟢 +1.2 🟢 +1.56 🟢 +0.76

Averages:

  Readability FRE GF ARI CLI DCRS
Average 43.26 34.96 11.66 14.85 14.34 8.53
  🟢 +0.01 🟢 +0 🟢 +0 🟢 +0 🟢 +0 🟢 +0
View metric targets
Metric Range Ideal score
Flesch Reading Ease 100 (very easy read) to 0 (extremely difficult read) 60
Gunning Fog 6 (very easy read) to 17 (extremely difficult read) 8 or less
Auto. Read. Index 6 (very easy read) to 14 (extremely difficult read) 8 or less
Coleman Liau Index 6 (very easy read) to 17 (extremely difficult read) 8 or less
Dale-Chall Readability 4.9 (very easy read) to 9.9 (extremely difficult read) 6.9 or less

Copy link

netlify bot commented Sep 5, 2024

Deploy Preview for n8n-docs ready!

Name Link
🔨 Latest commit ab97d9a
🔍 Latest deploy log https://app.netlify.com/sites/n8n-docs/deploys/66df0178efa2700008b71d29
😎 Deploy Preview https://deploy-preview-2455--n8n-docs.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 configuration.

Copy link
Contributor

@freakwriter freakwriter left a comment

Choose a reason for hiding this comment

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

@ivov Thanks for this PR. Can you review the two suggested commits I added? The first is just to make sure the intro includes the new endpoint. The second is to make our linting tool happier by replacing "does not" with "doesn't".

docs/hosting/logging-monitoring/monitoring.md Outdated Show resolved Hide resolved
docs/hosting/logging-monitoring/monitoring.md Outdated Show resolved Hide resolved
@ivov
Copy link
Contributor Author

ivov commented Sep 6, 2024

LGTM 👍🏻

@freakwriter freakwriter merged commit 5b40f5c into main Sep 10, 2024
7 checks passed
@freakwriter freakwriter deleted the readiness branch September 10, 2024 14:23
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.

2 participants