Skip to content

ENH: Update CircleCI external data cache#44

Merged
kwrobot merged 1 commit intoInsightSoftwareConsortium:masterfrom
blowekamp:CircleCIUseDataHash
Mar 23, 2018
Merged

ENH: Update CircleCI external data cache#44
kwrobot merged 1 commit intoInsightSoftwareConsortium:masterfrom
blowekamp:CircleCIUseDataHash

Conversation

@blowekamp
Copy link
Member

Introduce Yaml anchors to enable reuse of common steps.

The previous external data circleci cache was never being updated.

Obtain the most recent commit to modify a md5 or sha512 file. The hash
of this commit message is used for the hash key of the external data
CircleCI cache.

Change-Id: I29e35b15bfd3c3d2799ce835bd4a8dedd371f49a

See the CONTRIBUTING guide. Specifically:

Start ITK commit messages with a standard prefix (and a space):

  • BUG: fix for runtime crash or incorrect result
  • COMP: compiler error or warning fix
  • DOC: documentation change
  • ENH: new functionality
  • PERF: performance improvement
  • STYLE: no logic impact (indentation, comments)
  • WIP: Work In Progress not ready for merge

Provide a short, meaningful message that describes the change you made.

When the PR is based on a single commit, the commit message is usually left as the PR message.

A reference to a related issue or pull request in your repository. You can automatically close a related issues using keywords

@mentions of the person or team responsible for reviewing proposed changes.

Thanks for contributing to ITK!

Introduce Yaml anchors to enable reuse of common steps.

The previous external data circleci cache was never being updated.

Obtain the most recent commit to modify a md5 or sha512 file. The hash
of this commit message is used for the hash key of the external data
CircleCI cache.

Change-Id: I29e35b15bfd3c3d2799ce835bd4a8dedd371f49a
@blowekamp blowekamp force-pushed the CircleCIUseDataHash branch from d2bb50c to 76bc70e Compare March 23, 2018 13:36
@kwrobot kwrobot merged commit 76bc70e into InsightSoftwareConsortium:master Mar 23, 2018
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