Skip to content

SSE Exercise Add IO unit test and Github Actions automation#1

Open
aaronsch0 wants to merge 1 commit intomainfrom
extended-tests
Open

SSE Exercise Add IO unit test and Github Actions automation#1
aaronsch0 wants to merge 1 commit intomainfrom
extended-tests

Conversation

@aaronsch0
Copy link
Owner

This pull requests includes:

Test for the matrixIO::openData function by

  • creating a temporary CSV file
  • reading it back
  • verifying the matrix dimensions and values
  • cleaning up afterwards

Automation with github actions for

  • build
  • test
  • style

A workflow badge in the README.md file

Copy link
Contributor

@uekerman uekerman left a comment

Choose a reason for hiding this comment

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

Looks like you forgot to add some changes?

branches: [ "main", "master" ]

env:
BUILD_TYPE: Release
Copy link
Contributor

Choose a reason for hiding this comment

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

For testing, Debug is the better option.

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.

2 participants