Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support using clasp CLI to update apps script project #251

Open
simonkotwicz opened this issue Jan 18, 2022 · 4 comments
Open

Support using clasp CLI to update apps script project #251

simonkotwicz opened this issue Jan 18, 2022 · 4 comments
Labels
documentation enhancement New feature or request

Comments

@simonkotwicz
Copy link
Contributor

simonkotwicz commented Jan 18, 2022

I'm not sure what the current suggested way to update your own apps script project is (perhaps running uninstall on the current project, copying a new project, adding back customizations and then running install... or maybe just copying and pasting files?). I think maybe there should be a spot in the comments or README that specifies this. I don't mind making a PR to that effect.

However, in general I think it's easier to easier to pull down new releases/changes from github and push them to Google apps script with the clasp CLI. One of the issues with this right now is that when you pull down changes from a google apps script project it renames files with a .gs extension to use .js locally (and therefore creates a duplicate of each file with a different extension). So I was wondering if it might make sense to rename .gs files with the .js extension in this repo.

Another thing to make updates a bit cleaner would be to separate out the config section into its own file.

... So 3 proposed changes:

  1. Add update instructions (assuming they're not already somewhere)
  2. Rename .gs files to use .js
  3. Externalize personal config to a config.js or config.json file
@simonkotwicz simonkotwicz added the enhancement New feature or request label Jan 18, 2022
@artmg
Copy link

artmg commented Aug 30, 2023

The issue where claps push renames files clasp issue #51 has now been fixed. As of 2.4.2 the behaviour is definitely to leave the .gs extension when pushing local files into script files.
Only 1. and 3. would be required.

@artmg
Copy link

artmg commented Aug 30, 2023

Item 3. has been delivered via branch local-settings of my fork:

I have kept this separate in case anyone from the main project might be interested. Now I will work on a branch to address and document 1.

@jonas0b1011001
Copy link
Collaborator

I added clasp instructions to an upcoming release https://github.com/derekantrican/GAS-ICS-Sync/blob/settings-v2/About.gs, feel free to comment on it as i don't use this setup personally.

@artmg
Copy link

artmg commented May 29, 2024

Thanks for including those, I reckon they should be clear enough for anyone who is advanced enough to want to try clasp.

Thanks also for your efforts to rationalise and improve the project structure, I hope that the community gets the hang of it quickly, as it looks more logical to me - well done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants