Skip to content

Add contributor arnavsharma990 with interactive Tic-Tac-Toe component#34

Closed
arnavsharma990 wants to merge 1 commit intotypelevel:mainfrom
arnavsharma990:add-arnavsharma990
Closed

Add contributor arnavsharma990 with interactive Tic-Tac-Toe component#34
arnavsharma990 wants to merge 1 commit intotypelevel:mainfrom
arnavsharma990:add-arnavsharma990

Conversation

@arnavsharma990
Copy link
Copy Markdown

@arnavsharma990 arnavsharma990 commented Mar 8, 2026

  • Added a new contributor file: arnavsharma990.scala
  • Implemented an interactive Tic-Tac-Toe mini game using Calico and Cats Effect
  • Registered my handle in all.scala
  • Verified that the contributor order passes using the Check order button

Copilot AI review requested due to automatic review settings March 8, 2026 07:11
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new contributor component for arnavsharma990, featuring an interactive Tic-Tac-Toe mini-game built with Calico + Cats Effect, and registers the contributor in the global contributor list.

Changes:

  • Added arnavsharma990 contributor component implementing an interactive Tic-Tac-Toe game using SignallingRef state.
  • Registered arnavsharma990 in allContributors list.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/main/scala/gsoc/contributors/arnavsharma990.scala New contributor UI component with Tic-Tac-Toe game and required policy agreement statement.
src/main/scala/gsoc/contributors/all.scala Adds arnavsharma990 to the allContributors registry list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

import calico.html.io.{*, given}
import calico.syntax.*

object TicTacToe:
Copy link

Copilot AI Mar 8, 2026

Choose a reason for hiding this comment

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

TicTacToe is introduced as a public top-level object in the gsoc.contributors package, which unnecessarily expands the package namespace and increases the risk of name collisions with other contributor files. Consider moving the game model/helpers inside the arnavsharma990 contributor block (like other contributors do), or make the helper object explicitly private and uniquely named (e.g., private object ArnavTicTacToe) to keep helper code scoped to this component. (See examples of top-level helpers being private object ... in antoniojimeneznieto.scala:73, tanmay_008.scala:78, yummy_yums.scala:100.)

Suggested change
object TicTacToe:
private object TicTacToe:

Copilot uses AI. Check for mistakes.
@armanbilge
Copy link
Copy Markdown
Member

Thank you for agreeing to the Typelevel GSoC AI Policy, in which you committed to never use AI to generate descriptions for PRs.

You used Copilot to generate a description of your PR, in violation of our policy.

Therefore, I will close your PR at this time.

@armanbilge armanbilge closed this Mar 11, 2026
@typelevel typelevel locked and limited conversation to collaborators Mar 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants