Skip to content

Move autoharness_analyzer to scripts/ #350

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

carolynzech
Copy link

Move autoharness_analyzer to this repository so it can be community-owned and maintained.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

@carolynzech carolynzech changed the title Move autoharness_analyzer to scripts/ Move autoharness_analyzer to scripts/ May 7, 2025
@carolynzech carolynzech marked this pull request as ready for review May 8, 2025 15:28
@carolynzech carolynzech requested a review from a team as a code owner May 8, 2025 15:28
@carolynzech carolynzech marked this pull request as draft May 12, 2025 16:33
@carolynzech carolynzech marked this pull request as ready for review May 15, 2025 18:20
// and store which functions are safe and unsafe

// Number of columns in {crate_name}_scan_functions.csv.
const SCANNER_COLS: usize = 6;
Copy link
Member

Choose a reason for hiding this comment

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

So I guess we need to update Kani first (i.e., merge the subtree-update PR) and then this one (as the Kani version used in CI right now will have 5 columns only, and CI is failing accordingly).

Copy link
Member

Choose a reason for hiding this comment

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

With the assumption that #354 will go in first: this PR then needs to revert 4fc868e.

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