Skip to content

Bump Periplum engine to v0.3.2 (fix marker scatter) + Built-with-Periplum badge#9

Merged
cmehat merged 3 commits into
mainfrom
seo/meta-tags
Jun 22, 2026
Merged

Bump Periplum engine to v0.3.2 (fix marker scatter) + Built-with-Periplum badge#9
cmehat merged 3 commits into
mainfrom
seo/meta-tags

Conversation

@cmehat

@cmehat cmehat commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #7 (which is already merged). Two commits didn't make it in because they were pushed after #7 merged:

  • Bump engine v0.3.0 → v0.3.2 — picks up the opt-in longitude-unwrap fix (v0.3.1) so the protocol-city markers stay at their true positions instead of scattering across world-copies, plus other engine fixes.
  • Built-with-Periplum badge in the README.

No behavioural change beyond the engine bump.

Copilot AI review requested due to automatic review settings June 19, 2026 19:30

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.

Pull request overview

Bumps the Periplum engine version used by the GitHub Pages map to pick up upstream fixes (notably the marker scatter fix), and adds a “Built with Periplum” badge to the repository README.

Changes:

  • Update Periplum engine CDN reference from v0.3.0 to v0.3.2 in the published site.
  • Add a “Built with Periplum” badge under the main README heading.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
README.md Adds a Periplum badge to highlight the engine used by the project.
docs/index.html Updates the Periplum CDN URL to v0.3.2 for the map runtime.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/index.html Outdated
<script src="https://unpkg.com/leaflet@1.9.4/dist/leaflet.js"
integrity="sha256-20nQCchB9co0qIjJZRGuk2/Z9VM+kNiyxNV1lvTlZBo=" crossorigin=""></script>
<script src="https://cdn.jsdelivr.net/gh/periplum/periplum-core@v0.3.0/dist/periplum.js"></script>
<script src="https://cdn.jsdelivr.net/gh/periplum/periplum-core@v0.3.2/dist/periplum.js"></script>
@cmehat

cmehat commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

Good catch — valid and consistent with the SRI-pinned Leaflet assets, and especially relevant since the engine is pinned by a (mutable) tag.

Fixed: added integrity="sha384-…" + crossorigin="anonymous" to the Periplum engine <script> here, and applied the same across all Periplum consumers and the template. To keep this from going stale under the auto-upgrade model, the template's "Upgrade Periplum" workflow now recomputes the SRI hash whenever it bumps the engine version, so the pinned hash and the pinned tag always match.

@cmehat cmehat merged commit 0208e21 into main Jun 22, 2026
1 check 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.

2 participants