code-423n4 / code-423n4/code423n4.com

Move warden and team registration to another repo and include as git submodule

Open
#2,100 3 comments 5 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement roadmap Stand by
Dominant language
JavaScript
Stars
179
Forks
173
PR merge metrics
No merged PRs in 30d

Description

Problem/opportunity:

  • As we expand write access to the site to include more community support and process around onboarding new wardens, permissions get messier.
  • Having a 100% public warden application review process is different from how we handle other applications and may lead to awkward scenarios
  • Notifications of warden application activity adds a lot of noise to the website repo and website dev activity adds a lot of noise to the application review process

Desired outcome

  • Warden application reviewers have robust access to be able to review and merge applications without also having full write access to the website and other C4 data
  • Warden application reviewers are able to use GitHub to openly review and discuss applications that they are on the fence about.

Proposed solution:

  • separate private repo for warden applications and data (ie github.com/code-423n4/wardens)
  • import as submodule into site repo
  • use github actions to trigger website deploys when there is a new merge in wardens repo
  • create warden application review GitHub team to manage permissions
  • use private GitHub PR discussions to review warden applications

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the proposed private wardens repository, Git submodule integration, GitHub Actions deployment trigger, and GitHub team permissions described in the issue. Done means applications and review discussions are separated from the website repository while approved changes still trigger website deploys.

Written by the indexing model from the issue text.

Assessment

Tech stack
git, github-actions
Domain
ci-cd, devops, infrastructure
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.