docs: add workflow badges to README #4
Merged
Conversation
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
- Added Quality Gates workflow badge - Added PR Size workflow badge - Updated license badge to match frontend style - Removed redundant REUSE badge (checked in quality.yml) Aligns with other SecPal repositories.
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR enhances the README documentation by replacing the existing REUSE Compliance badge with workflow status badges and updating the license badge format. The changes improve visibility into the repository's CI/CD health while maintaining consistency with other SecPal repositories.
Key Changes:
- Added Quality Gates and PR Size workflow badges to display CI/CD status
- Updated license badge text and link to use AGPL v3+ format with external GNU license link
- Removed REUSE Compliance badge (functionality covered by Quality Gates workflow)
kevalyq
added a commit
that referenced
this pull request
Nov 2, 2025
* fix: remove --no-verify suggestion from error message Removed option 3 from pre-push error message as it violates Critical Rule #4: No Bypass policy. The only legitimate override is .preflight-allow-large-pr for exceptional cases with maintainer approval. Follow-up to #81 based on Copilot review feedback. * refactor: optimize CHANGELOG validation for consistency - Replace bash-specific [[ =~ ]] with POSIX-compliant case statement - Combine multiple grep -v calls into single grep -Ev with pattern group - Use grep -m 1 for early termination (performance improvement) - Fix sed range calculation (UNRELEASED_END is relative, not absolute) - Add clarifying comments about CHANGELOG_EXEMPT_PREFIXES and case sync - Improve warning message with specific line count and non-blocking note - Extract filter logic to reusable filter_changelog_content() function Aligns api/scripts/preflight.sh with optimizations applied to frontend, contracts, and .github repositories per Copilot review feedback.
kevalyq
added a commit
that referenced
this pull request
Nov 9, 2025
- Fix diff3 marker pattern: Change '||||||' to '||||||| ' (6→7 pipes + space) - Update CHANGELOG: Correct conflict marker pattern to '||||||| ' - Update docs table: Fix diff3 marker pattern with trailing space - Fix docs link: Change '../scripts/preflight.sh' to 'PREFLIGHT.md' Resolves 4 Copilot review comments. Nitpick #4 (copyright format) is correct as-is.
kevalyq
added a commit
that referenced
this pull request
Nov 9, 2025
* feat: add Git conflict marker detection - Add check-conflict-markers.sh script (95 lines) - Integrate CI workflow for automated checks - Document usage and troubleshooting - Scans for <<<<<<, =======, >>>>>>>, ||||||| markers - Prevents commits with unresolved merge conflicts - Update CHANGELOG.md * fix: resolve Copilot review comments in PR #124 - Fix diff3 marker pattern: Change '||||||' to '||||||| ' (6→7 pipes + space) - Update CHANGELOG: Correct conflict marker pattern to '||||||| ' - Update docs table: Fix diff3 marker pattern with trailing space - Fix docs link: Change '../scripts/preflight.sh' to 'PREFLIGHT.md' Resolves 4 Copilot review comments. Nitpick #4 (copyright format) is correct as-is. * docs: indent conflict marker examples in CHECK_CONFLICT_MARKERS.md Prevents false positives in conflict marker detection script by indenting the example markers in documentation.
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 Workflow Badges to README
This PR enhances the README by adding workflow badges for better visibility into the repository's health and status.
🎯 Changes
📊 Before / After
Before:
After:
✅ Alignment
Aligns with SecPal frontend and contracts repositories for consistent documentation.
Small documentation improvement! 📚