Skip to content

Changes for schema and removing config models - #35

Merged
gupichon merged 11 commits into
mainfrom
change-schema-config-models
Sep 9, 2026
Merged

Changes for schema and removing config models#35
gupichon merged 11 commits into
mainfrom
change-schema-config-models

Conversation

@TeresiaOlsson

@TeresiaOlsson TeresiaOlsson commented Sep 8, 2026

Copy link
Copy Markdown
Member

I have removed the config models in this package and added an entry point so the schemas can be discovered by the schema registry. See PR python-accelerator-middle-layer/pyaml#386.

Ruff was also very unhappy with things that were not covered by the pre-install commits for some reason so I also had to add many changes to the format for the ruff workflow to pass on github.

@TeresiaOlsson

Copy link
Copy Markdown
Member Author

The test currently fails because they are run against the last release of pyaml and not the main branch. Not sure what the best workflow is for that... Maybe we need a stable branch in this repository too so pushed to main can test against main in pyaml and pushes to stable can test against the latest release of pyaml.

@TeresiaOlsson

TeresiaOlsson commented Sep 9, 2026

Copy link
Copy Markdown
Member Author

The test currently fails because they are run against the last release of pyaml and not the main branch. Not sure what the best workflow is for that... Maybe we need a stable branch in this repository too so pushed to main can test against main in pyaml and pushes to stable can test against the latest release of pyaml.

I have temporarily changed the workflow so the tests are run against the pyaml main branch. It is the same as how it is in pyaml-oa-cs at the moment.

@gupichon

gupichon commented Sep 9, 2026

Copy link
Copy Markdown
Member

I have temporarily changed the workflow so the tests are run against the pyaml main branch. It is the same as how it is in pyaml-oa-cs at the moment.

Yes synchronize several project is still an issue. I have a proposal for that (we already discuss it, we should create a dev branch in every projects)

@gupichon
gupichon merged commit b1d3bc1 into main Sep 9, 2026
2 checks passed
@TeresiaOlsson
TeresiaOlsson deleted the change-schema-config-models branch September 9, 2026 14:51
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