using anything other than gs, js, or ts for fileExtension results in script file not being pushed

[imthenachoman]

Expected Behavior

If you set fileExtension to something other than gs, js, or ts it should work with both pulling and pushing.

It does work with pulling but not with pushing.

I think it is because of this code:

const defaultClaspignore = `# ignore all files…
**/**

# except the extensions…
!appsscript.json
!**/*.gs
!**/*.js
!**/*.ts
!**/*.html

# ignore even valid files if in…
.git/**
node_modules/**
`;

and

// Methods for retrieving dotfiles.
export const DOTFILE = {
  /**
   * Reads ignore.resolve() to get a glob pattern of ignored paths.
   * @return {Promise<string[]>} A list of file glob patterns
   */
  IGNORE: async () => {
    const ignorePath = config.ignore;
    const content =
      ignorePath && fs.existsSync(ignorePath) ? fs.readFileSync(ignorePath, FS_OPTIONS) : defaultClaspignore;

    return splitLines(stripBom(content)).filter((name: string) => name.length > 0);
  }

Because the default ignore does not include the custom fileExtension option. I know you could fix this by using your own ignore file but I'm thinking the defaultClaspignore should read the config file. No?

[imthenachoman]

For what it is worth, I am working on a fix.

[imthenachoman]

I think I fixed this.