Skip to content
This repository has been archived by the owner on Mar 26, 2018. It is now read-only.

Store filerev summary to file and strip summary paths #78

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

eversteini
Copy link

This feature offers two possibility:

  • storing the filerev summary to a file
  • stripping the src/dest paths of the filerev summary

Therefore two option parameters are supported:

  • summaryFilePath
  • stripPath

summaryFilePath defines the path, where the filerev summary should be stored.
stripPath can be used to strip the src/dest summary paths. The value can either be a string, or an object with the properties src and dest. If only a string is passed, this string is removed from all src and dest paths. If an object is passed, src and dest paths get treated individually.

Example:

options: {
      algorithm: 'md5',
      length: 8,
      summaryFilePath: './tasks/filerevsummary.json',
      stripPath: {
        src: 'assets',
        dest: '.tmp/public'
      }
    },

My use-case:
I implemented this feature because I needed the filerev summary in a view helper to provide the hashed source path when a view gets loaded. Therefore the summary had to be stored somewhere locally.

}
} catch (err) {
}
catch (err) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

} catch (err) {

@Strajk
Copy link

Strajk commented Apr 1, 2016

Hi, any updates on this? @eversteini @kevva

I found these features really useful and having them merged would be great 🙏

@thomasfr
Copy link

I would also like to have this feature merged. I am using the fork from @eversteini at the moment. @kevva ?

@eversteini
Copy link
Author

We are now working with this additional feature since more than a year and everything is working fine. It would be cool if this could be merged.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants