Skip to content

POC: Consolidate language_data into a monorepo structure - #58

Draft
prateek-dagar wants to merge 47 commits into
georgkrause:mainfrom
prateek-dagar:monorepo-poc
Draft

POC: Consolidate language_data into a monorepo structure#58
prateek-dagar wants to merge 47 commits into
georgkrause:mainfrom
prateek-dagar:monorepo-poc

Conversation

@prateek-dagar

Copy link
Copy Markdown

Hi @georgkrause,

This is a Draft PR showcasing a Proof of Concept (POC) for consolidating the language_data repository into the main langcodes repository as discussed in georgkrause/language_data#14.

What this POC implements:

  1. Zero Namespace Breakage: The code remains inside the language_data directory, meaning import language_data remains fully compatible with legacy codebases with zero changes.
  2. Preserved Git History: Imported language_data into a subdirectory (language_data/) using git subtree to preserve all historical commits, authors, and metadata.
  3. Unified Testing & CI: Configured the root pytest configuration and .github/workflows/ci.yml to automatically install the local subdirectory and run both test suites together.
  4. Independent Release Pipelines: Configured the CD workflow (.github/workflows/cd.yml) and setuptools_scm tag prefixes (language_data-v* vs v*) to allow publishing langcodes and language_data as separate, independent PyPI wheels.
  5. Independent Read the Docs builds: Created separate .readthedocs-langcodes.yaml and .readthedocs-languagedata.yaml configurations to support separate documentation pages and URLs without name clashes.

Let me know your feedback. I am happy to incorporate any changes!

Robyn Speer and others added 30 commits February 2, 2021 16:38
Add continous delivery configuration
* ci: Run tests in CI with PR reporting

* chore!: Bump minimum python version to 3.8

* test: Add missing dependencies

* chore: Add venv and test artifacts to gitignore

* test: Add new test for util function

* fix typo
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.

5 participants