Skip to content

Conversation

@wptestyyh
Copy link
Member

Type

  • Bug fix
  • Feature addition
  • Feature update
  • Documentation
  • Build Infrastructure

Side Effects

  • Breaking change
  • Non-functional change

Goals

Publish 0.5.2

Technical Details

None

Test Results

Build succeeds

Reviewer Focus

None

Future Work

None

Checklist

  • Build target all compiles cleanly.
  • clang-format and clang-tidy deltas produced no new output.
  • Newly added functions include doxygen-style comment block.

@wptestyyh wptestyyh requested a review from a team as a code owner June 27, 2025 20:51
@abeltrano
Copy link
Contributor

abeltrano commented Jun 27, 2025

@wptestyyh I noticed that v0.5.2 was already published. Are you going to delete that and re-do the release after this is merged?

@wptestyyh
Copy link
Member Author

@wptestyyh I noticed that v0.5.2 was already published. Are you going to delete that and re-do the release after this is merged?

nope, I don't plan to redo it. I publish it, validate the ingestion works on windows, then create this PR to merge the 0.5.2 version tag back to develop branch. Since I didn't see anything is wrong, I will keep it. Let me know if anything is wrong.

@wptestyyh wptestyyh closed this Jun 28, 2025
@wptestyyh wptestyyh reopened this Jun 28, 2025
@wptestyyh wptestyyh merged commit 1ffeafc into develop Jun 29, 2025
8 checks passed
@abeltrano
Copy link
Contributor

@wptestyyh I noticed that v0.5.2 was already published. Are you going to delete that and re-do the release after this is merged?

nope, I don't plan to redo it. I publish it, validate the ingestion works on windows, then create this PR to merge the 0.5.2 version tag back to develop branch. Since I didn't see anything is wrong, I will keep it. Let me know if anything is wrong.

Sorry, I missed this was merging the release branch into develop. All good!

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