Skip to content

Comments

Added warning to the Semaphore docs#983

Merged
vplasencia merged 1 commit intosemaphore-protocol:mainfrom
Hariking967:patch-1
May 19, 2025
Merged

Added warning to the Semaphore docs#983
vplasencia merged 1 commit intosemaphore-protocol:mainfrom
Hariking967:patch-1

Conversation

@Hariking967
Copy link
Contributor

@Hariking967 Hariking967 commented May 19, 2025

Added warning to the Semaphore docs

Description

Related Issue(s)

Other information

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have run yarn format and yarn lint without getting any errors
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Important

We do not accept minor grammatical fixes (e.g., correcting typos, rewording sentences) unless they significantly improve clarity in technical documentation. These contributions, while appreciated, are not a priority for merging. If there is a grammatical error feel free to message the team.

Added warning to the Semaphore docs
Copy link
Member

@vplasencia vplasencia left a comment

Choose a reason for hiding this comment

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

Looks great! Thank you very much @Hariking967!

@vplasencia vplasencia merged commit 57132a3 into semaphore-protocol:main May 19, 2025
5 checks passed
@gitpoap-bot
Copy link

gitpoap-bot bot commented May 19, 2025

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2025 Semaphore Contributor:

GitPOAP: 2025 Semaphore Contributor GitPOAP Badge

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.

@Hariking967
Copy link
Contributor Author

Amazing!

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.

Add warning to the Semaphore docs

2 participants