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

New release? #2909

Open
greateggsgreg opened this issue Apr 11, 2024 · 6 comments
Open

New release? #2909

greateggsgreg opened this issue Apr 11, 2024 · 6 comments

Comments

@greateggsgreg
Copy link

There's been a lot of improvements to the repository since the last release. I was wondering if there is any appetite for cutting another minor release which would be simplify the build processes for others that use this great library.

Thanks for everything you all do for the DF community

@simsong
Copy link
Member

simsong commented Apr 11, 2024

Thanks. I'm looking for ways to make the CI/CD process more visible here and to improve code coverage of the tests. Would you like to help?

@joachimmetz
Copy link
Contributor

@simsong maybe relevant #2428

@greateggsgreg
Copy link
Author

@simsong What do you imagine that looking like? Github Actions that execute the release/* scripts?

@simsong
Copy link
Member

simsong commented Apr 24, 2024

I was thinking of implementing it more-or-less the same way I did with bulk_extractor:

  • Special unit test targets that use a C/C++ unit test framework
  • Implemented as a GitHub action on pull-requests.
  • Having code coverage runs uploaded to codecov.io, so we can start creating tests that systematically cover the code base.
  • Run with GitHub Actions.

See:

@joachimmetz
Copy link
Contributor

Given this was discussed before in #2472 I would recommend to go ahead with the proposal. If you're going for GitHub actions, I recommend removing the travis and appveyor CI configurations if they are not used.

@simsong
Copy link
Member

simsong commented Apr 24, 2024

Given this was discussed before in #2472 I would recommend to go ahead with the proposal. If you're going for GitHub actions, I recommend removing the travis and appveyor CI configurations if they are not used.

Thank you for the recommendation. I have also discussed this with @bcarrier. I hope to continue the discussion at sleuthkit/governance#3

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

No branches or pull requests

3 participants