Skip to content
This repository was archived by the owner on Feb 15, 2024. It is now read-only.

GH-59 Allow to use custom instances of XMLInputFactory and XMLOutputFactory - #60

Merged
FrankSchnicke merged 20 commits into
admin-shell-io:developmentfrom
kenwenzel:custom-xml-factory
May 16, 2022
Merged

GH-59 Allow to use custom instances of XMLInputFactory and XMLOutputFactory#60
FrankSchnicke merged 20 commits into
admin-shell-io:developmentfrom
kenwenzel:custom-xml-factory

Conversation

@kenwenzel

@kenwenzelkenwenzel commented Mar 30, 2022

Copy link
Copy Markdown

This PR fixes the XML and AASX (de)serializers to correctly use custom instances of XMLInputFactory and XMLOutputFactory

Fixes issue #59

Daespenand others added 20 commits November 19, 2021 10:20
- Fixes branch names
- Adds section to mention CI workflows and their effects
…at as current AML serialization specification does not cover attribute introduces in model v3.0)
- added unit tests for simple example (serialization does currently loose information as as current AML serialization specification does not cover attribute introduces in model v3.0)
Is now triggered on release published
Signed-off-by: Frank Schnicke <frank.schnicke@iese.fraunhofer.de>
…ence-deserialization
Bugfix/reference deserialization
This is a dummy commit to trigger the workflows.
@CLAassistant

CLAassistant commented Mar 30, 2022

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@mjacoby

Copy link
Copy Markdown
Contributor

Looks fine to me.
@FrankSchnicke can you do the merge/release?

@arnoweiss

Copy link
Copy Markdown

This PR should be to development, I think.

@kenwenzel

Copy link
Copy Markdown
Author

development is actually 19 commits behind main, but feel free to change the base branch.
@FrankSchnicke Is there any work left from my side?

@arnoweiss
arnoweiss changed the base branch from main to developmentMay 10, 2022 08:59
@FrankSchnicke

Copy link
Copy Markdown
Contributor

I can gladly merge. However, before we do a release, I would like to clarify the overall process (cf. #61) to ensure that there's no wrong release.

Sign up for freeto 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.

7 participants

@kenwenzel@CLAassistant@mjacoby@arnoweiss@FrankSchnicke@Daespen@sebbader