search

sindresorhus / gulp-rev

Static asset revisioning by appending content hash to filenames: `unicorn.css` → `unicorn-d41d8cd98f.css`
MIT License
1.54k stars 217 forks source link

Parallel Tasks Overwriting Manfiest File #264

Open 23scurtu opened 3 years ago

23scurtu commented 3 years ago

No, locking is not the right answer. Do what @BerkeleyTrue said and you should be good :)

Originally posted by @bobthecow in https://github.com/sindresorhus/gulp-rev/issues/115#issuecomment-138101135

With regards to #115, why is locking the incorrect answer here? It seems unnecessarily error-prone to have parallel tasks possibly overwrite or garble the manifest file. Would it not make sense to have some form of locking for the manifest file, rather than to have undefined behaviour in a common use case and expect people to stumble upon #115 and implement some work around themselves?