Skip to content

Fix malformed URLs in JSON schema#8

Merged
fabricebrito merged 2 commits into
developfrom
copilot/sub-pr-7
Jan 21, 2026
Merged

Fix malformed URLs in JSON schema#8
fabricebrito merged 2 commits into
developfrom
copilot/sub-pr-7

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 21, 2026

Addresses review feedback on PR #7 regarding malformed URLs in schemas/config.schema.yaml.

Changes

  • Corrected $schema URL: https:/json-schema.org/...https://json-schema.org/...
  • Corrected $id URL: https:/www.terradue.com/...https://www.terradue.com/...

Both URLs were missing the second slash in the protocol scheme, preventing proper schema validation.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: fabricebrito <1178901+fabricebrito@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP Address feedback on 'Merge develop' PR changes Fix malformed URLs in JSON schema Jan 21, 2026
Copilot AI requested a review from fabricebrito January 21, 2026 14:30
@fabricebrito fabricebrito marked this pull request as ready for review January 21, 2026 14:31
@fabricebrito fabricebrito merged commit c0bc6f4 into develop Jan 21, 2026
1 check passed
@fabricebrito fabricebrito deleted the copilot/sub-pr-7 branch January 21, 2026 14:31
fabricebrito added a commit that referenced this pull request Jan 21, 2026
* reformat: edit structure

* delete redundant README

* delete requirements.txt

* feat: edit paths

* feat: edit paths

* feat: clean coding

* docs: update docs

* docs: update docs

* fix in bash-rc

* fixes skaffold config

* first commit

* ci: create pipy package

* fix: edit pyproject with correct Terradue email

* ci: create pipy package

* ci: remove test

* ci: add branch

* ci: add branch

* ci: edit package ci

* Update README.md with app-level description

* Update README.md with reorganisation of the charters and jhsingle-native vs jupyter-server based Dockerfiles

* Update README.md for formatting

* Update README.md with docker auth for images pulling

* Updated requirements.txt with psycopg2

* updated upload-artifact version from. 3 to 3.1.3

* Updated upload-artifact version

* Added postgresql-devel to the  Dockerfile

* Updated download-artifact version

* feat: edit hatch env path

* generalising the app definition in confg.yaml

* detail generalised sections

* docs: initial docs

* ESAEOEPCA-243 dedicated md for apphub config and its integration in mkdocs yml

* feat: edit generate-config sh refrence

* feat: update documentation

* Update setup.cfg

* Update setup.cfg

* new refinements + added of a generalised section of an appHub application

* docs: documenation for config generator

* docs: docs for finding kernel

* feat: add logging

* feat: algining for pypi push

* ci: edit package build

* chore: edit README refrence

* docs: add AppHub Configurator Application Package doc

* updating docs

* chore: bump version 0.0.5

* docs: edit example folder path

* ci: updating

* update version to avoid ci error

* update ci and bump new version 0.0.6

* CI + config-generator doc (ESAEOEPCA-236) (#5)

* ci: create pipy package

* fix: edit pyproject with correct Terradue email

* ci: create pipy package

* ci: remove test

* ci: add branch

* ci: add branch

* ci: edit package ci

* feat: edit hatch env path

* docs: initial docs

* feat: edit generate-config sh refrence

* feat: update documentation

* Update setup.cfg

* Update setup.cfg

* docs: documenation for config generator

* docs: docs for finding kernel

* feat: add logging

* feat: algining for pypi push

* ci: edit package build

* chore: edit README refrence

* docs: add AppHub Configurator Application Package doc

* updating docs

* chore: bump version 0.0.5

* docs: edit example folder path

* ci: updating

* update version to avoid ci error

* update ci and bump new version 0.0.6

---------

Co-authored-by: liviodonnini <47381409+liviodonnini@users.noreply.github.com>

* docs: edit intro

* docs: edit k8s doc

* feat: use apphub-configurator pypi pacakge in examples

* docs: improve documentation

* ci: ignore create new apphub-configurator

* chore: remove redundant files

* docs: edit configuration

* feat: bump a new version 0.0.7

* docs: edit configuration

* ci: trigger app-hub-config on develop and main

* docs: edit configuration

* ci: build pypi package when ci is changed

* docs: adding example to doc

* ci: copy example under docs

* feat: #377 initial implementation

* COPLACSER-445: fix: remove hardcoded namespace in dynamically generated ExternalSecret

* feat: #377 schema update according to the expected cases + improved task
to check multiple/single instance

* improvemt to manage namespace in dynamic way

* updated files to fix the issue with the gitlab-ci: https://github.com/EOEPCA/application-hub-context/actions/runs/15162303194/job/42631094503

* fixed pipeline issue

* fixed push on ghcr

* fixed push on ghcr

* fixed version for configurable-http-proxy

* feat: updated ignore list

* Add support for pvc annotations - bump to version 1.3.3 (#6)

* initial update

* minor fix to allow building the container

* updates README with container strategy

* Update Dockerfile

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update Dockerfile

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update .github/workflows/package.yaml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Fix malformed URLs in JSON schema file (#9)

* Initial plan

* Fix malformed URLs in config.schema.yaml

Co-authored-by: fabricebrito <1178901+fabricebrito@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: fabricebrito <1178901+fabricebrito@users.noreply.github.com>

* Fix malformed URLs in JSON schema (#8)

* Initial plan

* Fix malformed URLs in config schema - add missing colon

Co-authored-by: fabricebrito <1178901+fabricebrito@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: fabricebrito <1178901+fabricebrito@users.noreply.github.com>

* WIP on minilube configuration

* removes dev container, updates skaffold profiles

---------

Co-authored-by: parham-membari-terradue <parham.membari@terradue.com>
Co-authored-by: mlongobardo-gituname <mlongobardo>
Co-authored-by: Michele L. <37748899+xmichele@users.noreply.github.com>
Co-authored-by: fzingaretti <31729673+fzingaretti@users.noreply.github.com>
Co-authored-by: liviodonnini <47381409+liviodonnini@users.noreply.github.com>
Co-authored-by: Parham Membari <148858617+parham-membari-terradue@users.noreply.github.com>
Co-authored-by: stripodi <simone.tripodi@terradue.com>
Co-authored-by: ldonnini <livio.donnini@terradue.com>
Co-authored-by: Leonardo Mizzoni <leonardo.mizzoni@terradue.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: fabricebrito <1178901+fabricebrito@users.noreply.github.com>
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