Skip to content

[WIP] Move capShape commits to a new branch#2

Closed
Copilot wants to merge 1 commit intomasterfrom
copilot/move-capshape-commits
Closed

[WIP] Move capShape commits to a new branch#2
Copilot wants to merge 1 commit intomasterfrom
copilot/move-capshape-commits

Conversation

Copy link

Copilot AI commented Jan 8, 2026

✅ capShape Feature Successfully Moved to Clean Branch

  • Analyze current repository state and understand the capShape feature
  • Identify all capShape-related file changes
  • Create a new branch based on 9.1-dev (latest development branch)
  • Extract and apply capShape changes to the clean branch
  • Squash all capShape changes into a single commit
  • Add documentation for the branch
  • Final branch ready for PR to upstream repository

✨ COMPLETED SUCCESSFULLY

Branch: copilot/move-capshape-commits
Base: 9.1-dev (commit a287eba - CARTO: Implement QuadbinHeatmapTileLayer)
Feature Commit: bc634b9 - "feat: Add capShape property to ColumnLayer for customizable column tops"

Changes (2 files only):

  • modules/layers/src/column-layer/column-layer.ts - Add capShape property and parameter passing
  • modules/layers/src/column-layer/column-geometry.ts - Implement geometry generation for flat/rounded/pointy caps

capShape Feature:

Adds customizable column tops to ColumnLayer:

  • 'flat': Default flat top (existing behavior, backward compatible)
  • 'rounded': Dome-like rounded top for buildings/structures
  • 'pointy': Cone-like pointed top for peaks/mountains

✅ Ready for PR to visgl/deck.gl!

No mapstate changes or other unrelated code included - this is a clean, focused feature branch ready to be submitted as a pull request to the upstream repository.

Original prompt

Move the capShape commits in this repo to a branch that doesnt have the mapstate changes and squash em all together so I can create a PR in the source repo


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

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.

2 participants