Skip to content

[20411] Migrate to eProsima-CI#195

Merged
JLBuenoLopez merged 4 commits intomasterfrom
feature/migrate_to_eprosima_ci
Feb 14, 2024
Merged

[20411] Migrate to eProsima-CI#195
JLBuenoLopez merged 4 commits intomasterfrom
feature/migrate_to_eprosima_ci

Conversation

@EduPonz
Copy link
Copy Markdown

@EduPonz EduPonz commented Feb 12, 2024

This PR:

  • Refactors the build_and_test workflow so it becomes full multiplatform leveraging eProsima-CI
  • Fixes the CMake version used in CI
  • Adds a windows-2019 runner to the list of tests
  • Adds a Test summary step to the build_and_test workflow
  • Adds a Test reporting step to the build_and_test workflow
  • Correctly requires CMake 3.22 for testing

Depends on:

@EduPonz EduPonz force-pushed the feature/migrate_to_eprosima_ci branch from 62a3808 to 5487cea Compare February 12, 2024 15:19
@richiprosima
Copy link
Copy Markdown
Contributor

Build status:

  • Linux Build Status
  • Mac Build Status
  • Windows Build Status

@EduPonz EduPonz force-pushed the feature/migrate_to_eprosima_ci branch 5 times, most recently from 0b9947b to 1327f95 Compare February 12, 2024 20:09
@richiprosima
Copy link
Copy Markdown
Contributor

Build status:

  • Linux Build Status
  • Mac Build Status
  • Windows Build Status

@EduPonz EduPonz force-pushed the feature/migrate_to_eprosima_ci branch from 1327f95 to b918e14 Compare February 13, 2024 06:13
@richiprosima
Copy link
Copy Markdown
Contributor

Build status:

  • Linux Build Status
  • Mac Build Status
  • Windows Build Status

@EduPonz EduPonz changed the title [20441] Migrate to eProsima-CI [20411] Migrate to eProsima-CI Feb 13, 2024
@EduPonz EduPonz force-pushed the feature/migrate_to_eprosima_ci branch from 60d40a8 to 83dbd10 Compare February 13, 2024 10:23
@EduPonz EduPonz force-pushed the feature/migrate_to_eprosima_ci branch 3 times, most recently from 10aabee to 464e7bc Compare February 13, 2024 14:01
@EduPonz EduPonz marked this pull request as ready for review February 13, 2024 14:22
Copy link
Copy Markdown
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.

Changes LGTM. Pending approval when a release has been done in eProsima-CI including the required changes for this PR.

Comment thread .github/workflows/fastcdr-test.yml Outdated
@richiprosima
Copy link
Copy Markdown
Contributor

Build status:

  • Linux Build Status
  • Mac Build Status
  • Windows Build Status

Copy link
Copy Markdown
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

@JLBuenoLopez JLBuenoLopez merged commit c045c23 into master Feb 14, 2024
@JLBuenoLopez JLBuenoLopez deleted the feature/migrate_to_eprosima_ci branch February 14, 2024 06:04
@EduPonz
Copy link
Copy Markdown
Author

EduPonz commented Feb 14, 2024

@Mergifyio backport 1.1.x

@mergify
Copy link
Copy Markdown

mergify Bot commented Feb 14, 2024

backport 1.1.x

✅ Backports have been created

Details

mergify Bot pushed a commit that referenced this pull request Feb 14, 2024
* Refs #20423: Migrate to eProsima-CI and add Windows 2019 run

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Require CMake 3.22 for testing

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Add a test summary step

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Set v0 version for eProsima-CI actions

Signed-off-by: EduPonz <[email protected]>

---------

Signed-off-by: EduPonz <[email protected]>
(cherry picked from commit c045c23)

# Conflicts:
#	.github/workflows/fastcdr-test.yml
#	cmake/testing/GoogleTest.cmake
JesusPoderoso pushed a commit that referenced this pull request Feb 14, 2024
* Refs #20423: Migrate to eProsima-CI and add Windows 2019 run

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Require CMake 3.22 for testing

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Add a test summary step

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Set v0 version for eProsima-CI actions

Signed-off-by: EduPonz <[email protected]>

---------

Signed-off-by: EduPonz <[email protected]>
(cherry picked from commit c045c23)
EduPonz added a commit that referenced this pull request Feb 14, 2024
* Refs #20423: Migrate to eProsima-CI and add Windows 2019 run

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Require CMake 3.22 for testing

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Add a test summary step

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Set v0 version for eProsima-CI actions

Signed-off-by: EduPonz <[email protected]>

---------

Signed-off-by: EduPonz <[email protected]>
(cherry picked from commit c045c23)
Signed-off-by: EduPonz <[email protected]>
EduPonz added a commit that referenced this pull request Feb 14, 2024
* Migrate to eProsima-CI (#195)

* Refs #20423: Migrate to eProsima-CI and add Windows 2019 run

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Require CMake 3.22 for testing

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Add a test summary step

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Set v0 version for eProsima-CI actions

Signed-off-by: EduPonz <[email protected]>

---------

Signed-off-by: EduPonz <[email protected]>
(cherry picked from commit c045c23)
Signed-off-by: EduPonz <[email protected]>

* Include 1.1.x as CI target branch

Signed-off-by: JesusPoderoso <[email protected]>
Signed-off-by: EduPonz <[email protected]>

* Adjust to 1.1.x

Signed-off-by: EduPonz <[email protected]>

---------

Signed-off-by: EduPonz <[email protected]>
Signed-off-by: JesusPoderoso <[email protected]>
Co-authored-by: Eduardo Ponz Segrelles <[email protected]>
Co-authored-by: JesusPoderoso <[email protected]>
@EduPonz
Copy link
Copy Markdown
Author

EduPonz commented Feb 18, 2024

@Mergifyio backport 1.0.x

@mergify
Copy link
Copy Markdown

mergify Bot commented Feb 18, 2024

backport 1.0.x

✅ Backports have been created

Details

mergify Bot pushed a commit that referenced this pull request Feb 18, 2024
* Refs #20423: Migrate to eProsima-CI and add Windows 2019 run

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Require CMake 3.22 for testing

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Add a test summary step

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Set v0 version for eProsima-CI actions

Signed-off-by: EduPonz <[email protected]>

---------

Signed-off-by: EduPonz <[email protected]>
(cherry picked from commit c045c23)

# Conflicts:
#	.github/workflows/fastcdr-test.yml
#	cmake/testing/GoogleTest.cmake
EduPonz added a commit that referenced this pull request Feb 18, 2024
* Refs #20423: Migrate to eProsima-CI and add Windows 2019 run

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Require CMake 3.22 for testing

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Add a test summary step

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Set v0 version for eProsima-CI actions

Signed-off-by: EduPonz <[email protected]>

---------

Signed-off-by: EduPonz <[email protected]>
(cherry picked from commit c045c23)

# Conflicts:
#	.github/workflows/fastcdr-test.yml
#	cmake/testing/GoogleTest.cmake
Signed-off-by: EduPonz <[email protected]>
EduPonz added a commit that referenced this pull request Feb 19, 2024
* Migrate to eProsima-CI (#195)

* Refs #20423: Migrate to eProsima-CI and add Windows 2019 run

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Require CMake 3.22 for testing

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Add a test summary step

Signed-off-by: EduPonz <[email protected]>

* Refs #20423: Set v0 version for eProsima-CI actions

Signed-off-by: EduPonz <[email protected]>

---------

Signed-off-by: EduPonz <[email protected]>
(cherry picked from commit c045c23)

# Conflicts:
#	.github/workflows/fastcdr-test.yml
#	cmake/testing/GoogleTest.cmake
Signed-off-by: EduPonz <[email protected]>

* Adjust for 1.0.x

Signed-off-by: EduPonz <[email protected]>

* Apply José's suggestions

Signed-off-by: EduPonz <[email protected]>

---------

Signed-off-by: EduPonz <[email protected]>
Co-authored-by: Eduardo Ponz Segrelles <[email protected]>
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.

3 participants