Skip to content

Please do not reuse propertyType for "environmentVar " #248

Open

Description

@jkowalleck

Please do not reuse propertyType for "environmentVar".
The annotations from propertyType do not match the case here.

better define a dedicated (abstract) key-value pair element type:

<xs:complexType name="KeyValuePair">
    <xs:simpleContent>
        <xs:extension base="xs:string">
            <xs:attribute name="name" use="required">
                <xs:annotation><xs:documentation>The name of the env var</xs:documentation></xs:annotation>  
                <xs:simpleType>
                  <xs:restriction base="xs:token">
                    <xs:minLength value="1" />
                  </xs:restriction>
                </xs:simpleType>
            </xs:attribute>
        </xs:extension>
    </xs:simpleContent>
</xs:complexType>

(PS: this "KeyValuePair" is not identical to current implementation of "propertyType")

and use it here

<xs:element name="environmentVar" type="bom:KeyValuePair" />

Originally posted by @jkowalleck in #222 (comment)

Activity

  1. jkowalleck commented on Jun 16, 2023

    @jkowalleck
    MemberAuthor

    @stevespringett i do not know where to put it.
    milestone 1.6? 2.0?
    depends on the actual implementation, this could be a breaking change, or it could be a widening/expanding of existing types.

    it could be done via #249

  2. changed the title [-]Please do not reuse `propertyType` here.[/-] [+]Please do not reuse `propertyType` for "environmentVar "[/+] on Jun 16, 2023
  3. self-assigned this
    on Sep 8, 2026
  4. added this to the 2.0 milestone on Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions