Skip to content

Conversation

@adeyinkaoresanya
Copy link
Collaborator

Summary

This pull request introduces comprehensive architecture and product documentation for the CHAOSS Badging API. It focuses on:

  • Modularizing the codebase
  • Clearly defining requirements and workflows for both Event Badging and Project Badging
  • Ensuring maintainability and clarity for contributors

The PR includes:

  • An Architecture Decision Record (ADR) outlining the modular structure
  • Product Requirements Documents (PRDs) for Event Badging and Project Badging

These documents provide a single source of truth for workflow differences, separation of concerns, and future development guidelines.


Contents / Table of Contents

  1. Architecture Documentation and Modularization
  2. Product Requirements for Event Badging
  3. Product Requirements for Project Badging

1. Architecture Documentation and Modularization

  • File added: 001-adr-modular-badging-architecture.md

  • Key points:

    • Defines a modular, layered architecture separating routes, controllers, services, data access, and models
    • Introduces strict boundaries for Event vs Project Badging
    • Outlines shared infrastructure and cross-domain concerns
  • Impact:

    • Improves maintainability, readability, and contributor onboarding
    • Provides a decision record for future design changes

2. Product Requirements for Event Badging

  • File added: event-badging-PRD.md

  • Key points:

    • Reviewer-driven workflow: application submission, GitHub-based review, checklist handling, scoring, and badge awarding
    • Defines non-functional requirements
    • Clearly separates Event Badging from Project Badging processes
  • Impact:

    • Provides a clear reference for contributors implementing Event Badging
    • Reduces confusion and errors in workflow execution

3. Product Requirements for Project Badging

  • File added: project-badging-PRD.md

  • Key points:

    • Automated validation workflow for Project Badging, including repository scanning (DEI.md) and badge awarding logic
    • Ensures workflow separation from Event Badging
    • Includes non-functional requirements for reliability and maintainability
  • Impact:

    • Enables consistent, automated badge evaluation
    • Provides contributors with clear, actionable guidance

Reviewer Notes

  • All documents are interdependent; reviewing in context is recommended

  • Key areas to focus on:

    • ADR modular structure and boundaries
    • Accuracy and completeness of Event and Project workflows
    • Clarity for contributors unfamiliar with the codebase

Copy link
Contributor

@DesmondSanctity DesmondSanctity left a comment

Choose a reason for hiding this comment

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

LGTM

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