You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.
Cloud9 ace plugin reloads document when file on disk changes.
If diffing library is present, it can apply diff, to keep editor state like selection and folds.
But in case there is no diffing library, simply setting value is good enough.
Now adding dependency on diffing plugin, will break all configs using ace, and will require all of them to either include the diffing lib, or to include mock plugin as a fallback.
I think this is bad because it makes local change in ace plugin, into breaking change for all its consumers. But If optional dependencies were supported, only apps that needed diffing, would include it and it would only require one if statement in the ace plugin.
The text was updated successfully, but these errors were encountered:
usecase:
Cloud9 ace plugin reloads document when file on disk changes.
If diffing library is present, it can apply diff, to keep editor state like selection and folds.
But in case there is no diffing library, simply setting value is good enough.
Now adding dependency on diffing plugin, will break all configs using ace, and will require all of them to either include the diffing lib, or to include mock plugin as a fallback.
I think this is bad because it makes local change in ace plugin, into breaking change for all its consumers. But If optional dependencies were supported, only apps that needed diffing, would include it and it would only require one if statement in the ace plugin.
The text was updated successfully, but these errors were encountered: