pub: proof of personhood #22
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
madhavanmalolan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
WalkthroughA new non-technical contributor's guide was added to the README.md, providing explicit Git workflow instructions. The guide clarifies repository setup, dependency installation, branch management, and pushing changes. Additionally, a new blog post discussing Proof of Personhood and digital identity challenges was introduced in the content/posts directory. Changes
Sequence Diagram(s)sequenceDiagram
participant Contributor
participant GitHub Repo
participant Dev Team
Contributor->>GitHub Repo: Clone repository
Contributor->>GitHub Repo: Pull latest main branch
Contributor->>GitHub Repo: Create and switch to new branch
Contributor->>GitHub Repo: Commit and push changes
Contributor->>Dev Team: Notify for merge
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
Poem
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (5)
README.md (2)
14-22: Add language identifiers for all shell examples & drop prompt symbolsEach fenced block in this section is missing a language tag (
bash) and the$prompts violate MD014. Apply the same pattern as above to the “Pull latest” and “Create a branch” snippets to keep lint clean and improve readability.
73-75: Lint: fix final push snippetSame issue here—no language spec and stray
$.-``` -$ git push origin name_of_your_branch -``` +```bash +git push origin name_of_your_branch +```content/posts/data-flow copy.mdx (3)
17-21: Typos & punctuation
- “apporaches” → “approaches”
- Add the period after “etc.” to satisfy style rules.
-There are other apporaches too - that [uses your palm](https://www.humanity.org/), ... +There are other approaches too – that [use your palm](https://www.humanity.org/), ... ... -... like eating food, taking a cab etc. +... like eating food, taking a cab, etc.
32-38: Grammar & compound-adjective fixes
- Remove the article in “an impossible”
- Hyphenate “user-generated”
- Prefer “will” over “going to be” for certainty
- Period after “etc.”
-I feel it is an impossible to verify if an activity is done by a human on any app. +I feel it is impossible to verify whether an activity is done by a human on any app. -...digital services of user generated activity. +...digital services of user-generated activity. -...it is going to be hard to tell what activity was created by a human... +...it will be hard to tell which activity was created by a human... -...creating videos etc might actually be an AI agent... +...creating videos, etc., might actually be an AI agent...
52-56: Tone & formal wordingMinor style improvements for clarity and formality.
-...but sufficiently hard to discourage malicious webcrawlers and DDoS bots... +...but sufficiently difficult to discourage malicious web-crawlers and DDoS bots... -...could spin up a new account and continue till they get banned again. +...could spin up a new account and continue until they get banned again.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
README.md(2 hunks)content/posts/data-flow copy.mdx(1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
README.md
8-8: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
9-9: Dollar signs used before commands without showing output
(MD014, commands-show-output)
10-10: Dollar signs used before commands without showing output
(MD014, commands-show-output)
11-11: Dollar signs used before commands without showing output
(MD014, commands-show-output)
14-14: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
15-15: Dollar signs used before commands without showing output
(MD014, commands-show-output)
16-16: Dollar signs used before commands without showing output
(MD014, commands-show-output)
19-19: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
20-20: Dollar signs used before commands without showing output
(MD014, commands-show-output)
73-73: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
74-74: Dollar signs used before commands without showing output
(MD014, commands-show-output)
🪛 LanguageTool
content/posts/data-flow copy.mdx
[uncategorized] ~33-~33: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...of of personhood on digital services of user generated activity. I feel it is an impossible t...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
[style] ~36-~36: Use ‘will’ instead of ‘going to’ if the following action is certain.
Context: ...://huggingface.co/Qwen), it is clear it is going to be hard to tell what activity was creat...
(GOING_TO_WILL)
[style] ~36-~36: Try using a synonym here to add a more professional tone to your writing.
Context: ...co/Qwen), it is clear it is going to be hard to tell what activity was created by a ...
(BECOME_DIFFICULT)
[style] ~38-~38: In American English, abbreviations like “etc.” require a period.
Context: ...tivity - like tweeting, creating videos etc might actually be an AI agent using you...
(ETC_PERIOD)
[style] ~52-~52: To elevate your writing, try using a synonym here.
Context: .... It's not impossible, but sufficiently hard to discourage malicious webcrawlers and...
(HARD_TO)
[style] ~54-~54: Consider using the more formal “until”.
Context: ...ould spin up a new account and continue till they get banned again. Both of these ...
(TILL)
[style] ~72-~72: In American English, abbreviations like “etc.” require a period.
Context: ...ight block them too. Cloudflare, Google etc already do this. They share DDoS blockl...
(ETC_PERIOD)
| ``` | ||
| $ git@github.com:reclaimprotocol/blog.git | ||
| $ cd blog | ||
| $ npm install | ||
| ``` |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Incorrect Git clone command and missing fenced-block language spec
The command omits the clone verb and the block lacks a language hint, both of which break copy-paste UX and trigger MD040/MD014 lint errors.
-```
-$ git@github.com:reclaimprotocol/blog.git
-$ cd blog
-$ npm install
-```
+```bash
+git clone git@github.com:reclaimprotocol/blog.git
+cd blog
+npm install
+```🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
8-8: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
9-9: Dollar signs used before commands without showing output
(MD014, commands-show-output)
10-10: Dollar signs used before commands without showing output
(MD014, commands-show-output)
11-11: Dollar signs used before commands without showing output
(MD014, commands-show-output)
🤖 Prompt for AI Agents
In README.md around lines 8 to 12, the git clone command is missing the 'clone'
verb and the fenced code block lacks a language specifier, causing markdown lint
errors and poor copy-paste experience. Fix this by adding 'clone' to the git
command and specifying 'bash' as the language for the fenced code block,
ensuring the block starts with ```bash and ends with ```.
Description
Testing (ignore for documentation update)
Type of change
Checklist:
Additional Notes:
Summary by CodeRabbit