Skip to content

add copyright header Fix/797#826

Merged
mr-c merged 3 commits into
dib-lab:masterfrom
alameldin:fix/797
Feb 23, 2015
Merged

add copyright header Fix/797#826
mr-c merged 3 commits into
dib-lab:masterfrom
alameldin:fix/797

Conversation

@alameldin

Copy link
Copy Markdown
Contributor

fixes issue #797

@ged-jenkins

Copy link
Copy Markdown

Can one of the admins verify this patch?

1 similar comment
@ged-jenkins

Copy link
Copy Markdown

Can one of the admins verify this patch?

@alameldin

Copy link
Copy Markdown
Contributor Author
  • Is it mergeable?
  • Did it pass the tests?
  • If it introduces new functionality in scripts/ is it tested?
    Check for code coverage with make clean diff-cover
  • Is it well formatted? Look at make pep8, make diff_pylint_report,
    make cppcheck, and make doc output. Use make format and manual
    fixing as needed.
  • Did it change the command-line interface? Only additions are allowed
    without a major version increment. Changing file formats also requires a
    major version number increment.
  • Is it documented in the ChangeLog?
    http://en.wikipedia.org/wiki/Changelog#Format
  • Was a spellchecker run on the source code and documentation after
    changes were made?

@ctb

ctb commented Feb 23, 2015

Copy link
Copy Markdown
Member

Ok to test

Titus Brown, ctbrown@ucdavis.edu

On Feb 22, 2015, at 9:52 PM, ged-jenkins notifications@github.com wrote:

Can one of the admins verify this patch?


Reply to this email directly or view it on GitHub.

mr-c added a commit that referenced this pull request Feb 23, 2015
add copyright header Fix/797
@mr-c mr-c merged commit 0c8bc76 into dib-lab:master Feb 23, 2015
@mr-c

mr-c commented Feb 23, 2015

Copy link
Copy Markdown
Contributor

Thanks @alameldin

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.

4 participants