Skip to content

Add hrishabhxcode contributor component#37

Closed
hrishabhxcode wants to merge 3 commits intotypelevel:mainfrom
hrishabhxcode:add-hrishabhxcode-component
Closed

Add hrishabhxcode contributor component#37
hrishabhxcode wants to merge 3 commits intotypelevel:mainfrom
hrishabhxcode:add-hrishabhxcode-component

Conversation

@hrishabhxcode
Copy link
Copy Markdown

No description provided.

Copilot AI review requested due to automatic review settings March 9, 2026 06:57
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 interactive contributor component for the hrishabhxcode handle and registers it in the contributors list so it appears in the app.

Changes:

  • Introduces hrishabhxcode contributor UI with formula generation, copy-to-clipboard, and history.
  • Appends hrishabhxcode to the allContributors NonEmptyList.

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/hrishabhxcode.scala New contributor component implementation (formula generator + clipboard + history).
src/main/scala/gsoc/contributors/all.scala Registers the new contributor in the global contributor list.

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

Comment on lines +24 to +25
def copyToClipboard(text: String): IO[Unit] =
IO.delay(dom.window.navigator.clipboard.writeText(text))
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

copyToClipboard is typed as IO[Unit], but navigator.clipboard.writeText returns a JS Promise in Scala.js. IO.delay(...) will therefore produce an IO[js.Promise[Unit]] (and won’t sequence/fail the promise), which should not compile and won’t behave as intended. Convert the promise into IO[Unit] (e.g., use the cats-effect Promise interop and .void) and consider handling rejection so a clipboard permission failure doesn’t fail the click handler.

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.

Your PR description was generated by Copilot, 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