Skip to content

Fix to copyright licence check logic#487

Merged
pelson merged 2 commits into
SciTools:masterfrom
esc24:licence_fix
May 1, 2013
Merged

Fix to copyright licence check logic#487
pelson merged 2 commits into
SciTools:masterfrom
esc24:licence_fix

Conversation

@esc24

@esc24 esc24 commented May 1, 2013

Copy link
Copy Markdown
Member

This PR modifies the logic within the licence checking introduced by #477. It fixes an issue where the date of last change could be overwritten by an earlier modification.

@esc24

esc24 commented May 1, 2013

Copy link
Copy Markdown
Member Author

@pelson - would you mind taking a look?

@pelson

pelson commented May 1, 2013

Copy link
Copy Markdown
Member

I'd clearly misunderstood the results of git whatchanged - I thought I was getting one commit per file. This fixes the flaw in my logic on that front - thanks @esc24

pelson added a commit that referenced this pull request May 1, 2013
Fix to copyright licence check logic.
@pelson pelson merged commit 8aee327 into SciTools:master May 1, 2013
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.

2 participants