Dependency updates & repo reorganization - #209
Conversation
Signed-off-by: Emiliano Suñé <2395873+esune@users.noreply.github.com>
Signed-off-by: Emiliano Suñé <2395873+esune@users.noreply.github.com>
Signed-off-by: Emiliano Suñé <2395873+esune@users.noreply.github.com>
Signed-off-by: Emiliano Suñé <2395873+esune@users.noreply.github.com>
Signed-off-by: Emiliano Suñé <2395873+esune@users.noreply.github.com>
Signed-off-by: Emiliano Suñé <2395873+esune@users.noreply.github.com>
Signed-off-by: Emiliano Suñé <2395873+esune@users.noreply.github.com>
Signed-off-by: Emiliano Suñé <2395873+esune@users.noreply.github.com>
|
The @krobinsonca how can we verify this so we can clean things up further? |
There was a problem hiding this comment.
Pull request overview
This PR updates local/CI tooling versions and reorganizes OCA bundle content so bundles are grouped under organization folders (e.g., BC Registries, JAG, MDS), with corresponding registry and docs navigation updates.
Changes:
- Pin and install docs/schema tooling (MkDocs Material, AJV CLI) and add command-availability checks in DevContainer setup and CI workflows.
- Reorganize select BC Gov bundle folders (BC Registries, JAG, MDS) and update bundle registries (
ocabundles.json,ocabundleslist.json) accordingly. - Add/adjust bundle documentation files (READMEs, testdata, branding URLs) and update MkDocs navigation/docs references.
Reviewed changes
Copilot reviewed 19 out of 57 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/buildDevcontainer.sh | Adds pinned docs/schema tooling installs and post-setup command checks for DevContainer workflows. |
| ocabundleslist.json | Updates bundle registry list paths/checksums to match the reorganized bundle layout (plus adds JAG transcriber entries). |
| OCABundles/schema/bcgov-digital-trust/MDS/README.md | Adds top-level README for the new MDS grouping. |
| OCABundles/schema/bcgov-digital-trust/MDS/mines-act-permit/README.md | Adds README for the MDS Mines Act Permit bundle group. |
| OCABundles/schema/bcgov-digital-trust/MDS/mines-act-permit/1.1.1/README.md | Adds versioned README for Mines Act Permit 1.1.1. |
| OCABundles/schema/bcgov-digital-trust/MDS/mines-act-permit/1.1.1/OCABundle.json | Updates branding asset URLs to new MDS folder path. |
| OCABundles/schema/bcgov-digital-trust/MDS/mines-act-permit/1.1.1/branding.json | Updates branding asset URLs to new MDS folder path. |
| OCABundles/schema/bcgov-digital-trust/MDS/mines-act-permit/0.3/README.md | Adds versioned README for Mines Act Permit 0.3. |
| OCABundles/schema/bcgov-digital-trust/MDS/mines-act-permit/0.3/OCABundle.json | Updates branding asset URLs to new MDS folder path. |
| OCABundles/schema/bcgov-digital-trust/MDS/mines-act-permit/0.3/branding.json | Updates branding asset URLs to new MDS folder path. |
| OCABundles/schema/bcgov-digital-trust/JAG/transcriber-contractor/secondary_background.png | Adds/moves JAG transcriber bundle background image asset. |
| OCABundles/schema/bcgov-digital-trust/JAG/transcriber-contractor/README.md | Adds README for the JAG transcriber contractor bundle. |
| OCABundles/schema/bcgov-digital-trust/JAG/transcriber-contractor/OCABundle.json | Updates branding asset URLs to new JAG folder path. |
| OCABundles/schema/bcgov-digital-trust/JAG/transcriber-contractor/branding.json | Updates branding asset URLs to new JAG folder path. |
| OCABundles/schema/bcgov-digital-trust/JAG/transcriber-contractor/BCID_V_rgb_pos.png | Adds/moves JAG transcriber bundle logo asset. |
| OCABundles/schema/bcgov-digital-trust/JAG/README.md | Adds top-level README for the new JAG grouping. |
| OCABundles/schema/bcgov-digital-trust/bc-registries/README.md | Adds top-level README for the new BC Registries grouping. |
| OCABundles/schema/bcgov-digital-trust/bc-registries/business-card/testdata.csv | Adds example CSV data for the BC Registries business card bundle. |
| OCABundles/schema/bcgov-digital-trust/bc-registries/business-card/README.md | Adds README for the BC Registries business card bundle. |
| OCABundles/schema/bcgov-digital-trust/bc-registries/business-card-nonprod/testdata.csv | Adds example CSV data for the non-production business card bundle. |
| OCABundles/schema/bcgov-digital-trust/bc-registries/business-card-nonprod/README.md | Adds README for the non-production business card bundle. |
| OCABundles/schema/_archived/qc_anig_demo/README.md | Adds README for an archived QC ANIG demo grouping. |
| OCABundles/schema/_archived/qc_anig_demo/Attestation_numérique_didentité_gouvernemental/testdata.csv | Adds archived demo CSV data for QC ANIG bundle. |
| OCABundles/schema/_archived/qc_anig_demo/Attestation_numérique_didentité_gouvernemental/README.md | Adds README for archived QC ANIG bundle. |
| OCABundles/schema/_archived/qc_anig_demo/Attestation_numérique_didentité_gouvernemental/OCABundle.json | Adds archived QC ANIG demo OCA bundle JSON. |
| OCABundles/schema/_archived/qc_anig_demo/Attestation_numérique_didentité_gouvernemental/logo-quebec.png | Adds archived QC ANIG branding asset. |
| OCABundles/schema/_archived/qc_anig_demo/Attestation_numérique_didentité_gouvernemental/carte-anig-demo.png | Adds archived QC ANIG branding background asset. |
| OCABundles/schema/_archived/qc_anig_demo/Attestation_numérique_didentité_gouvernemental/branding.json | Adds archived QC ANIG branding overlay file. |
| OCABundles/schema/_archived/ditp-pilot/testdata.csv | Adds archived DITP pilot test CSV data. |
| OCABundles/schema/_archived/ditp-pilot/README.md | Adds README for archived DITP pilot bundle. |
| OCABundles/schema/_archived/ditp-pilot/OCABundle.json | Adds archived DITP pilot OCA bundle JSON. |
| OCABundles/schema/_archived/ditp-pilot/invitation-primary-background.png | Adds archived DITP pilot branding asset. |
| OCABundles/schema/_archived/ditp-pilot/invitation-logo.png | Adds archived DITP pilot branding asset. |
| OCABundles/schema/_archived/ditp-pilot/invitation-background-image-slice.png | Adds archived DITP pilot branding asset. |
| OCABundles/schema/_archived/ditp-pilot/branding.json | Adds archived DITP pilot branding overlay file. |
| ocabundles.json | Updates bundle registry map paths/checksums to match reorganization (adds JAG/MDS, updates BC Registries). |
| mkdocs.yml | Updates docs navigation to reflect new grouping folders and removes archived bundles from nav. |
| docs/README.md | Updates docs link to the new BC Registries bundle path. |
| docs/contributing/OCABundleCreatePR.md | Updates prerequisite list and adds DevContainer usage instructions for AJV/MkDocs. |
| docs/contributing/CONTRIBUTING.md | Links to the new AJV/MkDocs DevContainer instructions. |
| .github/workflows/validate.yml | Pins AJV/MkDocs versions and replaces file-existence action with a shell check; uses pinned AJV CLI. |
| .github/workflows/ci.yml | Pins MkDocs Material version and scopes cache keys to the version. |
| .devcontainer/devcontainer.json | Updates DevContainer image version and ensures PATH includes user bin folders and scripts; runs buildDevcontainer script. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
It should be transparent, we moved a few other bundles (showcase) already and they work as expected. |
Debian 12 (bookworm) marks system Python as externally managed (PEP 668), causing pip install to fail during devcontainer setup. Replace the direct pip install with a virtual environment at $HOME/.venv. Also export the updated PATH within the build script so the required_cmds verification loop succeeds without relying on .bashrc being sourced. Signed-off-by: Emiliano Suñé <2395873+esune@users.noreply.github.com>
Co-locates the devcontainer build script with devcontainer.json for better discoverability and conventional project layout. Signed-off-by: Emiliano Suñé <2395873+esune@users.noreply.github.com>
|
I think this is ready for review now. The move of the production |
krobinsonca
left a comment
There was a problem hiding this comment.
Looks good, we have this PR to fix some governance docs bcgov/DITP#154
Patch the two DITP governance links that point to OCA bundle paths moved by bcgov/aries-oca-bundles#209: - docs/governance/justice/court-services-branch/governance.md:183 transcriber-contractor moved under JAG/ - docs/governance/mining/bc-mines-act-permit/1.1.1/governance.md:180 mines-act-permit moved under MDS/ DO NOT MERGE until aries-oca-bundles#209 is merged into main in the bundles repo; the new paths must exist before this PR resolves. Signed-off-by: Kyle Robinson <kyle.robinson@briartech.ca>
This PR updates dependencies for tools used in this repository and reorganizes the bundles so that they are grouped in folders representing the issuing organization, rather than having everything sitting at the same level.
The
personfolder was deliberately left our for now as it will be moved underIASwith the updates to the related OCA bundle.This is an intermediate step to having a slightly more organized repo with the longer-term objective of transitioning from excel-based data sources to yaml or other plaintext file type.