Skip to content

Conversation

@CrackTheCode016
Copy link
Collaborator

πŸ“ Description

  • Added argument to the base chain-spec-generator command
  • Clarified what it does

πŸ” Review Preference

Choose one:

  • βœ… I have time to handle formatting/style feedback myself
  • ⚑ Docs team handles formatting (check "Allow edits from maintainers")

πŸ€– AI-Ready Docs

If content changed, regenerate AI files:

  • βœ… I ran python3 scripts/generate_llms.py
  • ⚑ Docs team will regenerate (check "Allow edits from maintainers")

βœ… Checklist

Copilot AI review requested due to automatic review settings October 23, 2025 19:33
@CrackTheCode016 CrackTheCode016 requested a review from a team as a code owner October 23, 2025 19:33
Copy link
Contributor

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

This PR documents the --chain-spec-path command-line flag for the chain-spec-builder tool, allowing users to specify a custom output directory for generated chain specifications.

Key Changes:

  • Added documentation for the --chain-spec-path flag usage with the chain-spec-builder create command

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
develop/parachains/deployment/generate-chain-specs.md Added example showing how to use --chain-spec-path flag to specify output location
llms-full.jsonl Updated AI training data with new documentation content
.ai/site-index.json Updated metadata timestamps and content hashes
.ai/pages/develop-parachains-deployment-generate-chain-specs.md Mirror of main documentation changes
.ai/categories/parachains.md Mirror of main documentation changes

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@CrackTheCode016 CrackTheCode016 added B0 - Needs Review Pull request is ready for review A3 - Quick Fixes Pull request contains small fixes (i.e., fixing a typo, removing old images, etc.) labels Oct 23, 2025
Copy link
Collaborator

@nhussein11 nhussein11 left a comment

Choose a reason for hiding this comment

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

Thank you @CrackTheCode016 for getting on this!!

Copy link
Collaborator

@dawnkelly09 dawnkelly09 left a comment

Choose a reason for hiding this comment

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

🚒 Thank you for making this update and helping to keep the docs fresh!

@dawnkelly09 dawnkelly09 added B1 - Ready to Merge Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Oct 27, 2025
@nhussein11 nhussein11 self-requested a review October 29, 2025 11:40
@nhussein11 nhussein11 merged commit 34fd0ee into master Oct 29, 2025
5 of 6 checks passed
@nhussein11 nhussein11 deleted the bd-fix-chainspec branch October 29, 2025 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A3 - Quick Fixes Pull request contains small fixes (i.e., fixing a typo, removing old images, etc.) B1 - Ready to Merge Pull request is ready to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants