-
Notifications
You must be signed in to change notification settings - Fork 2
Pr/website migration #435
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
Pr/website migration #435
Conversation
fixup: convert brand style guide to mdx and add to mission-control docs
…ch config, and vale setup
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
❌ Deploy Preview for flanksource-docs failed. Why did it fail? →
|
✅ Deploy Preview for canarychecker canceled.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements a website migration with comprehensive changes including:
- Vale linting configuration and results for documentation quality
- Multiple MDX-to-Markdown rendering scripts in different languages (Go, JavaScript)
- Style guide documentation and blog writing prompts
- Build tooling improvements using eget
- UI enhancements with new SVG assets and Tailwind configuration
Reviewed Changes
Copilot reviewed 205 out of 280 changed files in this pull request and generated 22 comments.
Show a summary per file
| File | Description |
|---|---|
| vale.json | Vale linting results for documentation files |
| styles/*.yml | Updated Vale style rules for spelling, acronyms, and complex words |
| scripts/render-for-context7/* | Multiple implementations of MDX rendering scripts |
| prompts/*.md | Writing style guidelines and blog post templates |
| modules/Makefile | Modernized build tooling with eget |
| mission-control/tailwind.config.js | Added CTA colors and gradient safelist |
| mission-control/static/img/* | New SVG illustrations |
| mission-control/src/pages/services/index.js | New services page component |
Files not reviewed (1)
- mission-control/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| 1. Avoid adverbs and complex language | ||
|
|
||
| ## Formating |
Copilot
AI
Nov 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected spelling of 'Formating' to 'Formatting'.
| 1. Avoid adverbs and complex language | ||
|
|
||
| ## Formating | ||
| - Format all output using MDX (markdowon) |
Copilot
AI
Nov 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected spelling of 'markdowon' to 'markdown'.
| @@ -0,0 +1,55 @@ | |||
| Write a 1250-1500 introductor blog post for a DevOps / Platform Engineering focused product or feature. | |||
Copilot
AI
Nov 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected spelling of 'introductor' to 'introductory'.
|
|
||
| Format the output in raw markdown suitable for copy and pasting into vscode. | ||
|
|
||
| Write a blog post on Flanksource MIssion Control approach to AIops, primarily building a real-time and update to mirror state of cloud resources that can be queried rapidly, plus an advanced graph that builds relationships between resources e.g. Cloudformation -> Auto Scaling Group > EC2 instance and then layers on soft relationsyhips like ebs volumes, subnets, IAM poilcies - For Kubernetes it understands Flux and Gitops being able to build a graph of a Flux Kustomization object creating a HelmRelease CRD, which then creates a deployment -> replicset -> pod and then layeying relationships like services, nodes, PVC, ingress, etc.. |
Copilot
AI
Nov 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Multiple spelling errors detected: 'MIssion' should be 'Mission', 'relationsyhips' should be 'relationships', 'poilcies' should be 'policies', 'replicset' should be 'replicaset', 'layeying' should be 'layering'.
|
|
||
| Write a blog post on Flanksource MIssion Control approach to AIops, primarily building a real-time and update to mirror state of cloud resources that can be queried rapidly, plus an advanced graph that builds relationships between resources e.g. Cloudformation -> Auto Scaling Group > EC2 instance and then layers on soft relationsyhips like ebs volumes, subnets, IAM poilcies - For Kubernetes it understands Flux and Gitops being able to build a graph of a Flux Kustomization object creating a HelmRelease CRD, which then creates a deployment -> replicset -> pod and then layeying relationships like services, nodes, PVC, ingress, etc.. | ||
|
|
||
| State based alerting (i.e. whene resource self-report failure) and traditioanl alerts from APM tools trigger playbooks that can then proactively collect infomation in a distrubuted fashion from agents deployed closest to the data, the graph, changes to the graph resources, events and pro-acrtive playboks are then fed into the model which tan the recommend futher playbooks to execute. |
Copilot
AI
Nov 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Multiple spelling errors: 'whene' should be 'when', 'traditioanl' should be 'traditional', 'infomation' should be 'information', 'distrubuted' should be 'distributed', 'pro-acrtive' should be 'proactive', 'playboks' should be 'playbooks', 'tan' should be 'can', 'futher' should be 'further'.
| import { | ||
| FaKubernetesIcon, | ||
| FaCloud, | ||
| FaShieldAlt, | ||
| FaCogs, | ||
| FaUsers, | ||
| FaRocket, | ||
| FaCheckCircle, | ||
| FaArrowRight, | ||
| FaPhone, | ||
| FaEnvelope, | ||
| FaSlack, | ||
| FaGithub, | ||
| FaDocker, | ||
| FaAws, | ||
| FaCloudsmith, | ||
| FaTools, | ||
| FaChartLine, | ||
| FaLifeRing, | ||
| FaExchangeAlt, | ||
| FaSearch, | ||
| FaLightbulb, | ||
| FaClock, | ||
| FaHeadset | ||
| } from 'react-icons/fa'; |
Copilot
AI
Nov 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unused imports FaAws, FaClock, FaCloud, FaCloudsmith, FaCogs, FaDocker, FaEnvelope, FaExchangeAlt, FaGithub, FaHeadset, FaKubernetesIcon, FaLifeRing, FaLightbulb, FaPhone, FaSearch.
| import { | ||
| FaCheck, | ||
| FaTimes, | ||
| FaUsers, | ||
| FaCloud, | ||
| FaShieldAlt, | ||
| FaEnvelope, | ||
| FaSlack, | ||
| FaPhoneAlt, | ||
| FaChevronDown, | ||
| FaChevronUp, | ||
| FaArrowRight, | ||
| FaDatabase, | ||
| FaChartLine, | ||
| FaBell, | ||
| FaServer, | ||
| FaLock, | ||
| FaLifeRing, | ||
| FaRocket, | ||
| FaQuestionCircle, | ||
| FaInfoCircle | ||
| } from 'react-icons/fa'; |
Copilot
AI
Nov 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unused imports FaCloud, FaEnvelope, FaLifeRing, FaPhoneAlt, FaShieldAlt, FaSlack, FaUsers.
| FaQuestionCircle, | ||
| FaInfoCircle | ||
| } from 'react-icons/fa'; | ||
| import { SiKubernetes, SiPrometheus, SiAmazonaws, SiMicrosoftazure, SiGooglecloud } from 'react-icons/si'; |
Copilot
AI
Nov 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unused imports SiAmazonaws, SiGooglecloud, SiKubernetes, SiMicrosoftazure, SiPrometheus.
| @@ -0,0 +1,18 @@ | |||
| import React from 'react'; | |||
| import Layout from '@theme/Layout'; | |||
| import CanaryCheckerPage from '@site/src/components/flanksource/CanaryCheckerPage'; | |||
Copilot
AI
Nov 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unused import CanaryCheckerPage.
| body: formDataToSubmit | ||
| }); | ||
|
|
||
| console.log(response) |
Copilot
AI
Nov 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Avoid automated semicolon insertion (94% of all statements in the enclosing function have an explicit semicolon).
| console.log(response) | |
| console.log(response); |
No description provided.