Skip to content

Conversation

rasmus-kirk
Copy link
Contributor

Purpose

See the related issue (BASE-558)

Checklist

  • My code follows the style of this project.
  • The code compiles without warnings.
  • I have performed a self-review of the changes.
  • I have documented my code, in particular the intent of the
    hard-to-understand areas.
  • (If necessary) I have updated the CHANGELOG.

@rasmus-kirk rasmus-kirk force-pushed the remove-test-infrastructure branch 2 times, most recently from 4c7c029 to 1838119 Compare April 8, 2025 09:34
@rasmus-kirk rasmus-kirk force-pushed the remove-test-infrastructure branch 12 times, most recently from 69431b8 to bbeb0a5 Compare April 8, 2025 14:30
@rasmus-kirk rasmus-kirk force-pushed the remove-test-infrastructure branch from bbeb0a5 to 0e8117b Compare April 8, 2025 14:33
@rasmus-kirk rasmus-kirk force-pushed the remove-test-infrastructure branch from 8c94bd8 to 897894c Compare June 10, 2025 11:38
@rasmus-kirk rasmus-kirk force-pushed the remove-test-infrastructure branch from 897894c to 2025dd9 Compare June 10, 2025 11:42
@rasmus-kirk rasmus-kirk force-pushed the remove-test-infrastructure branch from c4cd37f to ccf8500 Compare June 10, 2025 13:07
name: Test examples
runs-on: ubuntu-latest
needs: [rustfmt, clippy-schema]
#needs: [rustfmt, clippy-schema]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO

@allanbrondum
Copy link
Contributor

I think I need a better description of what is in this PR. It seems some stuff was removed. What is the reference for that? The linked issue does not seem to match with the changes. In addition, the PR includes an update of the Rust version. Is that related or unrelated?

@iumur iumur changed the title Remove test infrastructure COR-1515: Remove test infrastructure Aug 1, 2025
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