Skip to content

Add Quality Declaration for foonathan_memory external dependency [9398]#1404

Merged
MiguelCompany merged 5 commits intomasterfrom
feature/QDFoonathan
Sep 28, 2020
Merged

Add Quality Declaration for foonathan_memory external dependency [9398]#1404
MiguelCompany merged 5 commits intomasterfrom
feature/QDFoonathan

Conversation

@JLBuenoLopez
Copy link
Copy Markdown
Contributor

For more information about Foonathan API used by eprosima Fast DDS, please refer to #1403

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
@JLBuenoLopez JLBuenoLopez added the no-test Skip CI tests if PR marked with this label label Sep 23, 2020
@JLBuenoLopez JLBuenoLopez changed the title Add Quality Declaration for foonathan_memory external dependency Add Quality Declaration for foonathan_memory external dependency [9398] Sep 23, 2020
@imontesino imontesino self-requested a review September 23, 2020 12:44
…-aarch64 badges

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
imontesino
imontesino previously approved these changes Sep 23, 2020
@imontesino
Copy link
Copy Markdown
Contributor

LGTM

brawner
brawner previously approved these changes Sep 23, 2020
Copy link
Copy Markdown
Contributor

@brawner brawner left a comment

Choose a reason for hiding this comment

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

Looks good to me. Are you all planning on pushing this package to "quality level 1" equivalent?

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
@JLBuenoLopez JLBuenoLopez dismissed stale reviews from brawner and imontesino via 02ae1c5 September 24, 2020 11:58
Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany MiguelCompany merged commit 560d9ef into master Sep 28, 2020
@MiguelCompany MiguelCompany deleted the feature/QDFoonathan branch September 28, 2020 13:17
JLBuenoLopez added a commit that referenced this pull request Oct 23, 2020
* Refs #9391: Add Foonathan Quality Declaration

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

* Refs #9391: Update Foonathan QD

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

* Refs #9398: Add review suggestions by Ignacio Montesino. Change Linux-aarch64 badges

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

* Refs #9398: apply review comments by @brawner

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

* Apply suggestions from code review

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

Co-authored-by: Miguel Company <miguelcompany@eprosima.com>

Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
MiguelCompany added a commit that referenced this pull request Oct 23, 2020
* Added document with supported platforms (#1397)

* Refs #9263: added document with supported platforms

Signed-off-by: imontesino <monte.igna@gmail.com>

* Refs #9263: Applied suggested changes

Signed-off-by: imontesino <monte.igna@gmail.com>

* Refs #9263: Changed filename

Signed-off-by: imontesino <monte.igna@gmail.com>

* Refs #9263: removed windows arm32 reference

Signed-off-by: imontesino <monte.igna@gmail.com>

* Changed title

Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Add foonathan public API document (#1403)

* Refs #9394: Add foonathan public API document

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

* Update Public_API_foonathan_memory.md

Co-authored-by: Miguel Company <miguelcompany@eprosima.com>

Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Add CI labels to README

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

* Add Quality Declaration for foonathan_memory external dependency (#1404)

* Refs #9391: Add Foonathan Quality Declaration

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

* Refs #9391: Update Foonathan QD

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

* Refs #9398: Add review suggestions by Ignacio Montesino. Change Linux-aarch64 badges

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

* Refs #9398: apply review comments by @brawner

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

* Apply suggestions from code review

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

Co-authored-by: Miguel Company <miguelcompany@eprosima.com>

Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Add Quality Declaration and update README [9371] (#1400)

* Refs #9371: Add Quality Declaration and update README

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

* Refs #9371: correct typos

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

* Refs #9371: Update external dependencies

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

* Refs #9371: Update status summary

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

* Refs #9371: Update QD

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

* Refs #9371: Update QD, foonathan dependency

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

* Apply suggestions from code review

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

Co-authored-by: brawner <brawner@gmail.com>

Co-authored-by: brawner <brawner@gmail.com>
Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Fixed broken link to FastCDR Quality.md (#1443)

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

Co-authored-by: Ignacio Montesino Valle <ignamonte@hotmail.com>
Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
Co-authored-by: brawner <brawner@gmail.com>
Co-authored-by: Johnny Willemsen <jwillemsen@remedy.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-test Skip CI tests if PR marked with this label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants