Skip to content

ci: sanity check for prize submissions#43

Merged
siddarthkay merged 1 commit into
masterfrom
ci-check-for-submissions
May 12, 2026
Merged

ci: sanity check for prize submissions#43
siddarthkay merged 1 commit into
masterfrom
ci-check-for-submissions

Conversation

@siddarthkay
Copy link
Copy Markdown
Collaborator

@siddarthkay siddarthkay commented May 8, 2026

Summary

  • This is a level 1 check for basic mistakes that can be part of the prize submissions.
  • The github actions workflow will comment on the PR and will re-run when a PR is updated.

@siddarthkay siddarthkay self-assigned this May 8, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

❌ Validation failed

Fix before review:

  • ❌ Wrong title. Rename to Solution: LP-0099 <description>.
  • ❌ Solution PRs must only touch solutions/. Outside files: .github/scripts/validate-submission.sh .github/workflows/validate-submission.yml.
  • ❌ Missing sections in solutions/LP-0099.md: ## Summary, ## Repository, ## Approach, ## Success Criteria Checklist, ## FURPS Self-Assessment, ### Functionality, ### Usability, ### Reliability, ### Performance, ### Supportability, ## Terms & Conditions.
  • ❌ Missing repo link.
  • ❌ Missing T&C acknowledgment.
  • ❌ No success-criteria checklist. Mirror the criteria from the prize spec.
  • ❌ Prize LP-0099 not found in prizes/. Check the ID.

Warnings (non-blocking):

  • ⚠️ ### Functionality: only template prompt text. Fill in your assessment.
  • ⚠️ ### Usability: only template prompt text. Fill in your assessment.
  • ⚠️ ### Reliability: only template prompt text. Fill in your assessment.
  • ⚠️ ### Performance: only template prompt text. Fill in your assessment.
  • ⚠️ ### Supportability: only template prompt text. Fill in your assessment.

Automated check. See solution template and TERMS.

@siddarthkay siddarthkay force-pushed the ci-check-for-submissions branch 2 times, most recently from ab0d5a6 to 15ca791 Compare May 8, 2026 12:30
@siddarthkay
Copy link
Copy Markdown
Collaborator Author

@mart1n-xyz : good enough for basic PR sanity checks?

@siddarthkay
Copy link
Copy Markdown
Collaborator Author

I would remove 15ca791 and merge this PR and we can tweak this workflow as it reviews actual PR submissions

@mart1n-xyz
Copy link
Copy Markdown
Collaborator

Thanks, @siddarthkay. Please apply any changes if desired, and we're ready to merge.

@siddarthkay siddarthkay force-pushed the ci-check-for-submissions branch 2 times, most recently from c06fa42 to 40d7286 Compare May 12, 2026 14:37
@siddarthkay siddarthkay merged commit 40d7286 into master May 12, 2026
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.

3 participants