Skip to content

docs: fix proposal 7 deposit - #30

Merged
jhelison merged 1 commit into
mainfrom
docs/fix-prop-7
Jul 28, 2026
Merged

docs: fix proposal 7 deposit#30
jhelison merged 1 commit into
mainfrom
docs/fix-prop-7

Conversation

@jhelison

Copy link
Copy Markdown
Contributor

Description

Fix the proposal 7 deposit amount by removing one zero.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Documentation (updates documentation on the project)
  • chore (Updates on dependencies, gitignore, etc)
  • test (For updates on tests)

Copilot AI review requested due to automatic review settings July 28, 2026 20:36
@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The proposal deposit in kiichain/proposals/proposal_7.json was reduced from 35000000000000000000000akii to 3500000000000000000000akii.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description includes a summary and type, but omits the related issue, motivation, dependencies, and testing details. Add the related issue, motivation/context, any dependencies, and a testing section with reproducible verification steps.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the change to proposal 7's deposit amount.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/fix-prop-7

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Ready to approve

The change is a straightforward documentation/data correction and the resulting deposit value matches the convention used in the other proposal JSON files.

This review doesn't count toward merge requirements.

Pull request overview

This PR corrects the deposit amount in Kiichain’s Proposal 7 JSON by removing an extra zero, aligning it with the deposit format used across the other proposal files in kiichain/proposals.

Changes:

  • Update proposal_7.json deposit value from 35000000000000000000000akii to 3500000000000000000000akii.
File summaries
File Description
kiichain/proposals/proposal_7.json Fixes the proposal deposit amount by reducing it by one order of magnitude (removing one trailing zero).
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Low

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@kiichain/proposals/proposal_7.json`:
- Line 16: Update the "deposit" value in proposal_7.json to meet or exceed the
expedited governance minimum of 35000000000000000000000 akii, preserving the
proposal’s "expedited": true setting.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e98c03b9-6e52-4e3b-a3b3-c6dff08025b7

📥 Commits

Reviewing files that changed from the base of the PR and between 32e860e and 21b6cd8.

📒 Files selected for processing (1)
  • kiichain/proposals/proposal_7.json

Comment thread kiichain/proposals/proposal_7.json
@jhelison
jhelison merged commit 86f92ea into main Jul 28, 2026
2 checks passed
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.

3 participants