Skip to content

Conversation

zbeekman
Copy link
Collaborator

@zbeekman zbeekman commented Aug 9, 2017

Avg response time coverage on master
Issue Stats Codecov branch

Summary of changes

  • Prevent Makefiles from getting into release archives (advanced users only)
  • Added News section to README w/ content about trying OpenCoarrays online
  • Don't include gasnet dir in release archives

Rationale for changes

Save users from themselves & allow users to try OpenCoarrays in the cloud w/ zero setup/config etc

Additional info and certifications

This pull request (PR) is a:

  • Bug fix
  • Feature addition
  • Other, Please describe:

I certify that

  • I reviewed and followed the contributing guidelines, including
    - Increasing test coverage for all feature-addition PRs
    - Increasing test coverage for all bug-fix PRs for which there
    does not already exist a related test that failed before the PR
    - At least maintaining test coverage for all other PRs
    - Ensuring that all tests pass when run locally
    - Naming PR to indicate work in progress (WIP) and to attach the PR
    to the appropriate bug report or feature request issue
    - White space (no trailing white space or white space errors may
    be introduced)
    - Commenting code where it is non-obvious and non-trivial
    - Logically atomic, self consistent and coherent commits
    - Commit message content
    - Waiting 24 hours before self-approving the PR to give another
    OpenCoarrays developer a chance to review my proposed code

 - Only include officially supported CMake base build for end users
 - Of course this includes the "easy mode" shell script that wraps CMake
 - Exclude the src/gasnet dir from release archives because MPI is the current
   official backend. Only advanced users should try the gasnet version which
   is probably quite stale.
 - OpenCoarrays can be demoed in the cloud using binder.
 - See https://github.com/sourceryinstitute/jupyter-CAF-kernel for details
@zbeekman
Copy link
Collaborator Author

zbeekman commented Aug 9, 2017

LGTM

documentation changes only.

Approved with PullApprove Approved with PullApprove

@codecov
Copy link

codecov bot commented Aug 10, 2017

Codecov Report

Merging #432 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #432   +/-   ##
=======================================
  Coverage   39.38%   39.38%           
=======================================
  Files           3        3           
  Lines        2105     2105           
  Branches      350      350           
=======================================
  Hits          829      829           
  Misses       1154     1154           
  Partials      122      122

@zbeekman zbeekman merged commit 0198af6 into master Aug 10, 2017
@zbeekman zbeekman deleted the small-fixes branch August 10, 2017 01:21
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.

1 participant