Skip to content

ReconstructSPI Workflow Infrastructure#33

Open
thisFreya wants to merge 14 commits into
masterfrom
reconstructSPI_infrastructure
Open

ReconstructSPI Workflow Infrastructure#33
thisFreya wants to merge 14 commits into
masterfrom
reconstructSPI_infrastructure

Conversation

@thisFreya
Copy link
Copy Markdown
Collaborator

Files for Github workflow in reconstructSPI. Mostly adapted from ioSPI.

@thisFreya
Copy link
Copy Markdown
Collaborator Author

Tried to fix the codecov error but no dice - it should only be present in this commit as there is no actual code present.

Comment thread .codecov.yml
@geoffwoollard
Copy link
Copy Markdown

geoffwoollard commented Mar 31, 2022

Note that the tests were passing fine in #21

As soon as there is some code that gets tested, the test coverage will go up and a 0% test coverage error from init files won't arise.

Comment thread .codecov.yml
Comment thread .github/workflows/lint.yml
Comment thread environment.yml Outdated
Copy link
Copy Markdown

@geoffwoollard geoffwoollard left a comment

Choose a reason for hiding this comment

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

Remove dependencies. Also do we want to have a dev branch? To see if the testing/linting runs we have to merge into master. And then as something breaks, master is broken.

@thisFreya
Copy link
Copy Markdown
Collaborator Author

thisFreya commented Mar 31, 2022

Also do we want to have a dev branch? To see if the testing/linting runs we have to merge into master. And then as something breaks, master is broken.

@geoffwoollard Not sure what you mean by this, surely if the PR's have tests passing before they merge into master things shouldn't break?

@thisFreya
Copy link
Copy Markdown
Collaborator Author

Also do we want to have a dev branch? To see if the testing/linting runs we have to merge into master. And then as something breaks, master is broken.

Not sure what you mean by this, surely if the PR's have tests passing before they merge into master things shouldn't break?

@geoffwoollard I've added a branch "dev" to the requisite spots.

@geoffwoollard
Copy link
Copy Markdown

Also do we want to have a dev branch? To see if the testing/linting runs we have to merge into master. And then as something breaks, master is broken.

@geoffwoollard Not sure what you mean by this, surely if the PR's have tests passing before they merge into master things shouldn't break?

Yes yes good point! master won't break unless the failing tests in the PR are merged...

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.

3 participants