Repository navigation
docs: add FAQ and JSON-LD schema - #1
Merged
Merged
Conversation
🏗️ predictive-ml-core — architecture risk auditAnalysis of 4 system(s) for Acme complete. Overall score (weighted by criticality): 67/100. 4 finding(s), 4 recommendation(s).
Findings
Top recommendations
Generated by Predictive-ML-Core — deterministic, auditable rule scoring, method in METHODOLOGY.md. |
The example-usage workflow was pulling a pre-built image from Docker Hub, which caused the PR comment to contain Portuguese text from the old build. Building from the current checkout ensures the action always reflects the latest source, including the English translation committed in ca8cd49. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Tests were asserting Portuguese finding titles/descriptions that were translated in ca8cd49 — updated to match the English strings now in ArchitectureAnalyzer.cs. Dockerfile was missing -f net10.0 on dotnet publish, causing NETSDK1129 for multi-target projects. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…files Missed in ca8cd49: test data strings (category/title/description), assertions checking for Portuguese finding titles, and block comments in ApiEndpointTests, SeverityPredictionEngineTests, HtmlReportRendererTests, WebhookNotifierTests, ScoreBreakdownTests, and ArchitectureAnalyzerTests. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add FAQ section covering: what the lib does, vs ML.NET alone, .NET version, zero-allocation meaning, ASP.NET Core integration example.
Add JSON-LD FAQPage schema for LLM/search discoverability.