NWChemEx / .github

GitHub Settings for the NWChemEx Organization
https://nwchemex.github.io/.github/
Apache License 2.0
1 stars 2 forks source link

Separating Test and Build tasks to two actions #74

Closed quazirafi closed 1 year ago

quazirafi commented 1 year ago

PR Type

Brief Description

This PR separates build and test tasks into two separate actions.

Not In Scope

Anything not related to separating build and test actions.

PR Checklist

TODOs

Nothing.

quazirafi commented 1 year ago

You still have the .DS_store binary thing, which shouldn't be checked in.

Have you tested this PR and verified the tests run? Looking at .github/workflows/c-cpp_tmpl.yaml I don't see any calls to the testing action.

  1. I removed that .DS_store file. In my second last push it is showing that the file is removed.
  2. Yes, I tested for ParallelZone and it worked. See here please: https://github.com/NWChemEx-Project/ParallelZone/actions/runs/4160768371 @ryanmrichard
ryanmrichard commented 1 year ago

It's showing that it was renamed:

image

quazirafi commented 1 year ago

It's showing that it was renamed:

image

Can you check now? @ryanmrichard

quazirafi commented 1 year ago

Hopefully, it should be fine now. I also removed the header comments. @ryanmrichard

quazirafi commented 1 year ago

Let me test one more time then we can merge this.

quazirafi commented 1 year ago

It looks fine to me now. I just tested with ParallelZone again with all changes. @ryanmrichard

ryanmrichard commented 1 year ago

🚀 [bumpr] Bumped! New version:v0.0.2 Changes:v0.0.1...v0.0.2