Skip to content
This repository has been archived by the owner on Mar 10, 2023. It is now read-only.

evaluate the $ref early during the loading #225

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

goneri
Copy link
Member

@goneri goneri commented Dec 20, 2022

Evaluate and get ride of all the $ref just after the loading of
the JSON definition. This way we don't have to deal with the
references and definitions later during the processing.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ build-ansible-collection SUCCESS in 5m 11s
✔️ tox-cloud-generate-vmware SUCCESS in 3m 26s
✔️ ansible-tox-py38 SUCCESS in 4m 19s
✔️ ansible-tox-py39 SUCCESS in 4m 18s
ansible-tox-linters FAILURE in 4m 00s

Evaluate and get ride of all the $ref just after the loading of
the JSON definition. This way we don't have to deal with the
references and definitions later during the processing.
@goneri goneri force-pushed the evaluate-the-ref-early-during-the-loading_20126 branch from b954271 to 79d9292 Compare December 20, 2022 21:30
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ build-ansible-collection SUCCESS in 5m 11s
✔️ tox-cloud-generate-vmware SUCCESS in 3m 33s
ansible-tox-py38 FAILURE in 4m 15s
✔️ ansible-tox-py39 SUCCESS in 4m 19s
✔️ ansible-tox-linters SUCCESS in 4m 02s

@goneri
Copy link
Member Author

goneri commented Dec 20, 2022

recheck

@goneri goneri closed this Dec 20, 2022
@goneri goneri reopened this Dec 20, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ build-ansible-collection SUCCESS in 6m 01s
tox-cloud-generate-vmware FAILURE in 3m 31s
ansible-tox-py38 FAILURE in 4m 21s
✔️ ansible-tox-py39 SUCCESS in 4m 28s
✔️ ansible-tox-linters SUCCESS in 4m 12s

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant