Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enhancement: add information on property updates to the change recorder #880

Closed
andrzejj0 opened this issue Dec 24, 2022 · 1 comment · Fixed by #881
Closed

Enhancement: add information on property updates to the change recorder #880

andrzejj0 opened this issue Dec 24, 2022 · 1 comment · Fixed by #881
Labels
enhancement
Milestone
2.15.0

Comments

@andrzejj0
Copy link
Contributor

andrzejj0 commented Dec 24, 2022 •
edited
Loading

Change recorder only records dependency updates. In this enhancement I'll add property updates. This implies a new version of the XSD and a new namespace:

<schema xmlns="http://www.w3.org/2001/XMLSchema"
        elementFormDefault="qualified"
        xmlns:u="http://www.mojohaus.org/versions-maven-plugin/schema/updates/2.0"
        targetNamespace="http://www.mojohaus.org/versions-maven-plugin/schema/updates/2.0">

    <complexType name="VersionChangeType"/>

    <complexType name="DependencyVersionChangeType">
        <complexContent>
            <extension base="u:VersionChangeType">
                <attribute name="kind" type="string" use="required"/>
                <attribute name="groupId" type="string" use="required"/>
                <attribute name="artifactId" type="string" use="required"/>
                <attribute name="oldVersion" type="string" use="required"/>
                <attribute name="newVersion" type="string" use="required"/>
            </extension>
        </complexContent>
    </complexType>

    <complexType name="PropertyVersionChangeType">
        <complexContent>
            <extension base="u:VersionChangeType">
                <attribute name="property" type="string" use="required"/>
                <attribute name="oldValue" type="string" use="required"/>
                <attribute name="newValue" type="string" use="required"/>
            </extension>
        </complexContent>
    </complexType>

    <element name="updates">
        <complexType>
            <sequence maxOccurs="unbounded">
                <choice>
                    <element name="dependencyUpdate" type="u:DependencyVersionChangeType"/>
                    <element name="propertyUpdate" type="u:PropertyVersionChangeType"/>
                </choice>
            </sequence>
        </complexType>
    </element>

</schema>
@andrzejj0 andrzejj0 mentioned this issue Dec 24, 2022
Merged
@andrzejj0
Copy link
Contributor Author

Target release: 2.15.0.

andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Dec 24, 2022
@andrzejj0
Resolves mojohaus#880: add information on property updates to the cha…
729fd67 
…nge recorder

Release notes:
- new version and new namespace for the change recorder
- change recorder also records property changes
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Jan 4, 2023
@andrzejj0
Resolves mojohaus#880: add information on property updates to the cha…
9d480b6 
…nge recorder

Release notes:
- new version and new namespace for the change recorder
- change recorder also records property changes
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Jan 4, 2023
@andrzejj0
Resolves mojohaus#880: add information on property updates to the cha…
120d680 
…nge recorder

Release notes:
- new version and new namespace for the change recorder
- change recorder also records property changes
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Jan 4, 2023
@andrzejj0
Resolves mojohaus#880: add information on property updates to the cha…
c997bb4 
…nge recorder

Release notes:
- new version and new namespace for the change recorder
- change recorder also records property changes
@slawekjaranowski slawekjaranowski linked a pull request Jan 4, 2023 that will close this issue
Resolves #880: add information on property updates to the change recorder #881
Merged
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Jan 8, 2023
@andrzejj0
Resolves mojohaus#880: add information on property updates to the cha…
114ee9a 
…nge recorder

Release notes:
- new version and new namespace for the change recorder
- change recorder also records property changes
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Jan 8, 2023
@andrzejj0
Resolves mojohaus#880: add information on property updates to the cha…
da7fe18 
…nge recorder

Release notes:
- new version and new namespace for the change recorder
- change recorder also records property changes
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Jan 8, 2023
@andrzejj0
Resolves mojohaus#880: add information on property updates to the cha…
67374d9 
…nge recorder

Release notes:
- new version and new namespace for the change recorder
- change recorder also records property changes
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Jan 25, 2023
@andrzejj0
Resolves mojohaus#880: add information on property updates to the cha…
227aeed 
…nge recorder

Release notes:
- new version and new namespace for the change recorder
- change recorder also records property changes
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Feb 2, 2023
@andrzejj0
Resolves mojohaus#880: add information on property updates to the cha…
50367b5 
…nge recorder

Release notes:
- new version and new namespace for the change recorder
- change recorder also records property changes
slawekjaranowski pushed a commit that referenced this issue Feb 12, 2023
@andrzejj0 @slawekjaranowski
Resolves #880: add information on property updates to the change reco…
2c22da0 
…rder

Release notes:
- new version and new namespace for the change recorder
- change recorder also records property changes
@slawekjaranowski slawekjaranowski added this to the 2.15.0 milestone Feb 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
2.15.0
2 participants
@andrzejj0 @slawekjaranowski