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.

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

[20822] Remove cat from windows colcon build action #105

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

JesusPoderoso
Copy link
Contributor

@JesusPoderoso JesusPoderoso commented Jun 11, 2024

Description

The windows colcon build action has a cat colcon.meta command that prints in the terminal the given colcon meta (I assume that it was introduced for debugging purposes).
As long as some changes have been introduced in the Fast DDS CI to use different colcon.meta files at the same time (feature allowed by the --metas argument), this cat command does not recognize the meta file if more than one are provided.
This PR removes the cat.

Related PR:

Contributor Checklist

  • Commit messages follow the company guidelines.
  • Changes are API compatible.
  • New features have been added to the versions.md and README.md files (if applicable).

Reviewer Checklist

  • The title and description correctly express the PR's purpose.
  • The Contributor checklist is correctly filled.

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
@JesusPoderoso JesusPoderoso merged commit ec349b3 into main Jun 11, 2024
1 check passed
@JesusPoderoso JesusPoderoso deleted the hotfix/20822 branch June 11, 2024 08:56
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.

2 participants