ci: update dist-tag to make flags optional.

dgp1130 · alan-agius4 · commit 07802c9158c9 · 2020-07-23T10:45:25.000+02:00
We can't guarantee that the user provided these options at compile time, so all options should include `undefined` in their types. This makes the type system a little smarter and hopefully prevent future mistakes. Fortunately we were already doing the right thing, this will make sure we continue to do so in the future.

I opted to use `Partial&lt;DistTagArgs&gt;` so that all future flags would be implicitly forced to be optional. Otherwise, we may forget to make them optional and potentially introduce a bug.
diff --git a/scripts/dist-tag.ts b/scripts/dist-tag.ts
@@ -37,7 +37,7 @@ interface DistTagOptions {
 /**
  * This function adds a tag to all public packages in the CLI repo.
  */
-export default function(args: DistTagOptions, logger: logging.Logger) {
+export default function(args: Partial<DistTagOptions>, logger: logging.Logger) {
   if (args.help) {
     logger.info(`dist-tag adds a tag to all public packages in the CLI repo.
 
