Skip to content

[REFACTOR] Rename -core package to -common to remove a source of confusion#522

Merged
Seth Fitzsimmons (sethfitz) merged 1 commit into
devfrom
core2common
May 13, 2026
Merged

[REFACTOR] Rename -core package to -common to remove a source of confusion#522
Seth Fitzsimmons (sethfitz) merged 1 commit into
devfrom
core2common

Conversation

@sethfitz
Copy link
Copy Markdown
Collaborator

Description

As discussed and agreed in the 2026-03-11 / CODING SESH 💻 meeting, this change renames the overture-schema-core package to overture-schema-common.

The purpose of this change is to eliminate a source of confusion caused by overloading the word "core":

  • Since early days, the word "core" has been used in Overture to mean that part of the map and its schema that is of central importance and not peripheral.
  • However, in the overture-schema-core package, the word "core" was being used in a different sense to mean basically shared types that are common to more than one theme schema.

This issue has produced real confusion and consumed many minutes of unproductive discussion in meetings. It is hoped this change will eliminate the confusion going forward.

This recreates #520, which was merged but did not land on dev. Rebased onto current dev (which includes the entry-points discovery refactor in #463), so discovery references in this commit resolve to overture.schema.system.discovery rather than the moved-and-renamed overture.schema.common.discovery.

Related

Testing

$ make check

Signed-off-by: schapper <schapper@amazon.com>
@github-actions
Copy link
Copy Markdown

🗺️ Schema reference docs preview is live!

🌍 Preview https://staging.overturemaps.org/schema/pr/522/schema/index.html
🕐 Updated May 13, 2026 15:42 UTC
📝 Commit 247ff85
🔧 env SCHEMA_PREVIEW true

Note

♻️ This preview updates automatically with each push to this PR.

@sethfitz Seth Fitzsimmons (sethfitz) merged commit 8fd4534 into dev May 13, 2026
18 of 20 checks passed
@sethfitz Seth Fitzsimmons (sethfitz) deleted the core2common branch May 13, 2026 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename overture-schema-core package to overture-schema-common

4 participants