Skip to content

929: Changed author size test - #893

Closed
Arshadul-Monir wants to merge 1 commit into
mainfrom
929
Closed

929: Changed author size test#893
Arshadul-Monir wants to merge 1 commit into
mainfrom
929

Conversation

@Arshadul-Monir

@Arshadul-Monir Arshadul-Monir commented Apr 11, 2026

Copy link
Copy Markdown
Member

Description of changes

Since new Codebloom devs need to add their names to the /api endpoint, the assertion has been changed to check if authors.size() in the api controller is > 0 rather than strictly 6

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

Staging

@Arshadul-Monir

Copy link
Copy Markdown
Member Author

/deploy

@Arshadul-Monir
Arshadul-Monir enabled auto-merge (rebase) April 11, 2026 19:00
@github-actions

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

auto-merge was automatically disabled April 11, 2026 19:18

Pull request was closed

@Arshadul-Monir
Arshadul-Monir deleted the 929 branch April 11, 2026 19:18
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.

1 participant