Skip to content

docs: fill gaps in v5→v6 migration guide - #8227

Merged
mizmay merged 4 commits into
maplibre:mainfrom
mizmay:docs/v6-migration-guide-gaps
Aug 25, 2026
Merged

mizmay merged 4 commits into
maplibre:mainfrom
mizmay:docs/v6-migration-guide-gaps

Conversation

@mizmay

@mizmay mizmay commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Made added to the guide to where there was something to add that could either (a) better document the specific failure mode a developer might encounter or (b) highlight something they can do to verify or address it.

  • CommonJS removal: got more specific and included the error message one might encounter
  • @ latest 404: what you will encounter if you have @ latest set and specifically what to do instead
  • WebGL2 requirement and map.transform removal: included even though they aren't strictly migration tasks to highlight a specific action mentioned in the CHANGELOG and perhaps worth repeating here.

From #8168: deliberately not addressing the vite callout (already fixed in the vite docs so nothing to address) and the peer-range guidance for library authors (doesn't fit here).

Planning a follow-up migration skill in maplibre-agent-skills maybe just a proof of concept, but who knows, maybe it's useful to someone.

Launch Checklist

  • Confirm you have read our AI policy here.

Assisted-By:

Claude Sonnet 5.0 for research and testing to verify the failure modes, etc.

Comment thread docs/guides/v5-to-v6-migration-guide.md Outdated

@HarelM HarelM left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

THANKS! Added a minor comment.

@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.23%. Comparing base (062d745) to head (f7d3061).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8227      +/-   ##
==========================================
- Coverage   94.23%   94.23%   -0.01%     
==========================================
  Files         291      291              
  Lines       25030    25030              
  Branches     6610     6610              
==========================================
- Hits        23587    23586       -1     
- Misses       1443     1444       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@CommanderStorm CommanderStorm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

minor changes

Comment thread docs/guides/v5-to-v6-migration-guide.md Outdated
Comment thread docs/guides/v5-to-v6-migration-guide.md Outdated
@mizmay
mizmay requested a review from CommanderStorm August 24, 2026 22:53
@HarelM

HarelM commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

Feel free to squash-merge this when you see fit.

@mizmay
mizmay merged commit f67ec04 into maplibre:main Aug 25, 2026
25 checks passed
@mizmay
mizmay deleted the docs/v6-migration-guide-gaps branch August 25, 2026 05:57
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