[web] Statics dumpy??? #2

Emi · 2021-10-30T19:20:57Z

Emi commented

2021-10-30 19:20:57 +00:00

Implement pronouns-today dump-statics [path]
Defaults to ./static/
Should handle most common error cases pretty well
I made a whole branch for this 'cause I thought it would be more work, but now I'm committed

- Implement `pronouns-today dump-statics [path]` - Defaults to `./static/` - Should handle most common error cases pretty well - I made a whole branch for this 'cause I thought it would be more work, but now I'm committed

Emi added 1 commit 2021-10-30 19:20:58 +00:00

[web] Implement dumping of the static assets e1aca0497e

Emi requested review from sashanoraa 2021-10-30 19:21:07 +00:00

sashanoraa reviewed 2021-11-02 17:08:19 +00:00

web/src/statics.rs

					
				@ -52,0 +82,4 @@

								.map_err(|e| (e, path))

						)

						.collect::<Result<(), (io::Error, PathBuf)>>()

				}

sashanoraa commented

2021-11-02 17:08:19 +00:00

I think this might be cleaner with a for loop instead of 2 iterators but it's fine if you prefer this way.

Emi commented

2021-11-05 20:14:39 +00:00

The problem with a for loop is that we want to make sure that there are no conflicts before we start to write out files. If we used a for loop, it already would've written several files before it noticed the conflict. By using disjoint interators, we can create a "checkpoint" and make sure there are no errors first