Merge pull request #319 from sourceryinstitute/issue-318

Fix bad arg order in CMake command Fixes #318
sourceryinstitute · Jan 25, 2017 · 1b0627a · 1b0627a
2 parents 4a74633 + 1776e85
commit 1b0627a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/.pullapprove.yml b/.pullapprove.yml
@@ -11,19 +11,20 @@ groups:
         - master
         - devel
         - caffeinate-opencoarrays
-      files:
-        exclude:
-          - "*.md"
-    users:
-      - afanfa
-      - rouson
-      - zbeekman
+    teams:
+      - team-opencoarrays
         # https://github.com/orgs/sourceryinstitute/teams/team-opencoarrays
   OpenCoarrays:
     # Whenever someone has a PR, on any branch require at least 1 
     # collaborator to explicitly review it
     required: 1
     users: all
+    conditions:
+      branches:
+        exclude:
+          - master
+          - devel
+          - caffeinate-opencoarrays
   ConfigManagers:
     required: 1
     conditions:

diff --git a/src/tests/integration/pde_solvers/navier-stokes/CMakeLists.txt b/src/tests/integration/pde_solvers/navier-stokes/CMakeLists.txt
@@ -9,9 +9,9 @@ if ( ("${CMAKE_SYSTEM_PROCESSOR}" MATCHES "x86_64") AND ("${CMAKE_SYSTEM_NAME}"
       set(fft_library ${CMAKE_CURRENT_SOURCE_DIR}/libfft_avx.a )
     endif()
     add_executable( coarray_navier_stokes
+      EXCLUDE_FROM_ALL
       coarray-shear_coll.F90
       ${walltime_o}
-      EXCLUDE_FROM_ALL
     )
     target_link_libraries(coarray_navier_stokes OpenCoarrays ${fft_library})
   endif()