Chanseok Oh

> Also, the `jib-classpath-file` file does not reference this shared source set. Just a minor point: `jib-classpath-file` is for the classpath used to launch a JVM in the container. It...

We introduced the above blank App Engine Deployment page above #1592, and in fact, there have been discussions if it's a good idea to show this all the time for...

I am on to this. This requies refactoring code from the standard Maven-based wizard, and will be fixed when I unify the two standard wizards.

Oh, are you saying setting the group ID from the package or the other way around? I was talking about the latter, which we currently do in the Maven-based std...

> ... returns most-up-to-date preferences values (e.g., users updated the preferences while a deploy is in progress) rather than the values at the time when the instance is constructed by...

The main motivation for setting a "storage path", as I see it, is to be able to persist datastore and indexes across server runs. (As such, I think there is...

> A request to [be able to specify `--skip_sdk_update_check`](https://stackoverflow.com/questions/44134717/why-is-an-exception-thrown-when-trying-to-check-for-updates-to-the-google-app-en). It's not clear if the user is actually using CT4E. (He says he's doing _Run As_ > _Web Application_.) It's kind...

``` DeployCommandHandler --+-- StandardDeployCommandHandler | +-- FlexDeployCommandHandler | +-- FlexExistingArtifactDeployCommandHandler ``` I think `FlexExistingArtifactDeployCommandHandler` will soon inherit `FlexDeployCommandHandler` to fix #2438. They will share some code.

I may be missing something, but not sure if we need this. `web.xml` validation already works, even without our CT4E (probably even without WTP). These XSDs are publicly available online,...

Still in that case, wouldn't it be enough to specify online URLs rather than paths to local files?