Skip to content

810: update styleguide#832

Merged
luoh00 merged 1 commit into
mainfrom
810
Mar 5, 2026
Merged

810: update styleguide#832
luoh00 merged 1 commit into
mainfrom
810

Conversation

@luoh00

@luoh00 luoh00 commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

810

Description of changes

updated styleguide by adding extra guidelines for frontend naming conventions and themes

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

image image

Staging

@github-actions

github-actions Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Available PR Commands

  • /ai - Triggers all AI review commands at once
  • /review - AI review of the PR changes
  • /describe - AI-powered description of the PR
  • /improve - AI-powered suggestions
  • /deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

@github-actions

github-actions Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions

github-actions Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

@luoh00

luoh00 commented Mar 4, 2026

Copy link
Copy Markdown
Contributor Author

/deploy

@luoh00

luoh00 commented Mar 4, 2026

Copy link
Copy Markdown
Contributor Author

/ai

@tahminator

Copy link
Copy Markdown
Member

/review

@tahminator

Copy link
Copy Markdown
Member

/describe

@tahminator

Copy link
Copy Markdown
Member

/improve

@github-actions

github-actions Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

@github-actions

github-actions Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Preparing PR description...

@github-actions

github-actions Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@luoh00

luoh00 commented Mar 4, 2026

Copy link
Copy Markdown
Contributor Author

/deploy

@luoh00 luoh00 merged commit 1f81690 into main Mar 5, 2026
47 of 48 checks passed
@luoh00 luoh00 deleted the 810 branch March 5, 2026 00:56
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