Skip to content

Conversation

@adriancampo
Copy link
Contributor

@adriancampo adriancampo changed the base branch from main to feature/xtypes1.3 February 12, 2024 14:00
@adriancampo adriancampo force-pushed the feature/typeidentifier_comparator branch from 51276ce to fab20d4 Compare February 12, 2024 14:01
Copy link
Contributor

@JLBuenoLopez JLBuenoLopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two small nitpicks

Copy link
Contributor

@JLBuenoLopez JLBuenoLopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a typo

Copy link
Contributor

@JLBuenoLopez JLBuenoLopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. This PR needs to wait before being considered for merging.

JLBuenoLopez and others added 4 commits February 15, 2024 15:40
* Refs #19906: add test case: structure inheritance with intermediate empty structure

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs #19906: remove unused header

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs #19906: extend Annotations.idl test cases

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs #19906: fix typo

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs #20129: minor fixes to pass TypeObject tests

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs #20129: fix inherited bitset

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs #20129: apply review suggestions

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

---------

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
Signed-off-by: adriancampo <adriancampo@eprosima.com>
Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
richiware and others added 3 commits February 16, 2024 12:12
* Refs #17138. New template for PR

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>

* Refs #17138. Apply suggestion

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>

---------

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>
* Refs #20424: complete structure members

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs #20424: fix union typo

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs #20424: use correctly wide strings

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs #20424: add structure using substructures with same name

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

---------

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
* Refs #19451. Rebase and remove CRLR

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>

* Refs #19451. Add .gitattributes

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>

* Refs #19451. Apply suggestion

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>

---------

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>
@adriancampo adriancampo force-pushed the feature/typeidentifier_comparator branch 2 times, most recently from fa0a545 to da37d6f Compare March 22, 2024 11:05
adriancampo and others added 3 commits March 25, 2024 09:48
Signed-off-by: adriancampo <adriancampo@eprosima.com>
Signed-off-by: adriancampo <adriancampo@eprosima.com>
Signed-off-by: adriancampo <adriancampo@eprosima.com>
@adriancampo adriancampo force-pushed the feature/typeidentifier_comparator branch from da37d6f to eb5b513 Compare March 25, 2024 09:23
Signed-off-by: adriancampo <adriancampo@eprosima.com>
@adriancampo adriancampo force-pushed the feature/typeidentifier_comparator branch from a4503b3 to 2defa4b Compare March 25, 2024 11:30
Base automatically changed from feature/xtypes1.3 to main June 11, 2024 13:39
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.

4 participants