User bio
404 bio not found
Member since Dec 10, 2015
Posts:
Michael has not published any posts yet.
Replies:

Hi Brett

I have re-installed the old version and the option description said:

"Automatically modify the class name or ROUTINE header to match the file's path when copying or creating a new file."

So we definitely need it to set the routine header when creating a new file and we think it should be the default like it was before.

Thanks.

Michael

Hi @Brett Saviano 

We have been doing further testing and have come across an issue. In our environment, we have various git hooks setup to configure certain IRIS globals when classes change. Without this, the compile will fail for some generated classes.

What happens now is, when you git pull, this extension detects the file system changes and attempts to compile the classes before the git hook has completed, causing errors to be reported. Looking at the logs, I can see VSCode attempted to compile a few seconds before the git hook completed.

To resolve this, please would you consider adding a "Seconds to Delay Local Sync for External Changes" setting? We could then set it to 10 or whatever and it would wait that amount of time after a file system change before attempting a compile.

There is no need to delay the compile when saving directly in VS Code so the setting should either only be for external changes, or have two delay settings, one for internal and another for external. We'd be happy with just a delay for external changes.

Thanks.

Michael

I have been testing these changes and like them very much, especially the monitoring of the file system which means discards/stashes via external tools now work (code is recompiled).

However, I'm not a fan of this change:

Change the default value of the objectscript.autoAdjustName setting to false. Now that we have an index of the workspace, we no longer require that a document's name match the file path for the extensions to find it.

It means when creating a brand new objectscript routine it is now missing the header and therefore invalid. It took me a while to find the setting and re-enable it. Did you really mean to stop new routines from auto inserting a valid header?

Certifications & Credly badges:
Michael has no Certifications & Credly badges yet.
Global Masters badges:
Michael has no Global Masters badges yet.
Followers:
Michael has no followers yet.
Following:
Michael has not followed anybody yet.