Skip to content

docs(base-sync): align deepslate header and registry with stone contract (#240) - #251

Merged
BharathASL merged 2 commits into
mainfrom
docs/deepslate-base-sync-contract
Oct 8, 2026
Merged

BharathASL merged 2 commits into
mainfrom
docs/deepslate-base-sync-contract

Conversation

@BharathASL

Copy link
Copy Markdown
Contributor

Description

Align the deepslate base-sync header and registry entry with the stone contract (#240). Documents that derivatives copy the (currently empty) block along with , enforces single-line formatting of sharedSections in tools/base-sync.json, and adds unit test coverage for header wording, formatting invariants, and missing detection.

Affected Assets

  • textures/block/deepslate.svg

Type of Change

  • feat: New vector texture master, blockstate, or compiler capability
  • fix: Tiling fix, palette correction, or bugfix
  • docs: Documentation improvement
  • chore / refactor: Maintenance, dependencies, or codebase cleanup

Contributor Checklist

Please check all applicable boxes before requesting review:

Vector Texture Standards (if adding/modifying SVGs)

  • Authored as a clean $512\times512$ SVG (viewBox="0 0 512 512").
  • Primary shapes are aligned to the $32\text{px}$ texel grid ($16\times16$ grid).
  • Seamless Toroidal Tiling: Checked and verified that elements wrapping across $X$ and $Y$ edges align perfectly without seams or chopped shapes.
  • Ore Consistency: If adding an ore texture, the stone background geometry and corner radius (rx) match textures/block/stone.svg exactly.
  • Free of leftover AI generation comments, unnecessary editor namespaces, or embedded raster images.

Build & Verification

  • Ran npm run build locally and verified that the pack compiles successfully.
  • Verified textures in-game or inspected the rasterized PNG outputs in dist/.

Git Hygiene & Standards

  • Commit message(s) follow Conventional Commits (e.g., feat(textures): ...).
  • Every commit is signed off with the Developer Certificate of Origin (git commit -s).
  • Branch is rebased cleanly onto latest main with no merge commits.
  • Formatting complies with .editorconfig (2-space indent, LF endings, trailing newline).

…act (#240)

Align deepslate base-sync header and registry entry with the stone contract and enforce via test suite.

Signed-off-by: Bharath <bharathasl74185@gmail.com>
…al space

Require horizontal whitespace only in sharedSections formatting check.

Signed-off-by: Bharath <bharathasl74185@gmail.com>

@ninja6-agent ninja6-agent 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.

No issues found.

@BharathASL
BharathASL added this pull request to the merge queue Oct 8, 2026
Merged via the queue into main with commit a8b3034 Oct 8, 2026
5 checks passed
@BharathASL
BharathASL deleted the docs/deepslate-base-sync-contract branch October 8, 2026 07:14
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.

1 participant