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

Lower the bar on building Bio-Routing #459

Merged
merged 5 commits into from
Jan 7, 2024

Conversation

BarbarossaTM
Copy link
Member

@BarbarossaTM BarbarossaTM commented Jan 2, 2024

This is intended as the first step to lower the bar of entry on using Bio-Routing :-)

  • Add a Makefile to build Bio-Routing commands and examples & test the code
  • Update GitHub workflows to use make targets
  • Update README to point out how to build and run bio-rd
  • Some clean-ups

A good next step would be to start populating Documentation/user a bit more on more example configurations and how to use bio-rd and bio-rdc together.

  This Makefile provides some useful targets:
   * build (default)
   * clean
   * test
   * tets-coverage (test + write coverage)
   * all (clean build test)

Signed-off-by: Maximilian Wilhelm <max@sdn.clinic>
Signed-off-by: Maximilian Wilhelm <max@sdn.clinic>
  Remove build_examples.sh script replaced by Makefile

Signed-off-by: Maximilian Wilhelm <max@sdn.clinic>
@BarbarossaTM BarbarossaTM added type:improvement Add robustness/improvements type:clean-up Code clean-up labels Jan 2, 2024
@BarbarossaTM BarbarossaTM self-assigned this Jan 2, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jan 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c567987) 52.14% compared to head (6b9d920) 52.14%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #459   +/-   ##
=======================================
  Coverage   52.14%   52.14%           
=======================================
  Files         159      159           
  Lines        8384     8384           
=======================================
  Hits         4372     4372           
  Misses       3789     3789           
  Partials      223      223           
Flag Coverage Δ
unittests 52.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@BarbarossaTM BarbarossaTM changed the title Building Lower the bar on building Bio-Routing Jan 2, 2024
Signed-off-by: Maximilian Wilhelm <max@sdn.clinic>
Signed-off-by: Maximilian Wilhelm <max@sdn.clinic>
@ricky-charlet
Copy link

Thanks. These would be very helpful to me and others approaching this project from scratch.

@taktv6 taktv6 merged commit a330f47 into bio-routing:master Jan 7, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:clean-up Code clean-up type:improvement Add robustness/improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants