Skip to content

Commit

Permalink
Make Linter Extension and API public (WIP) (#13902)
Browse files Browse the repository at this point in the history
* Make Linter base case public

* Typos

* Missing base class to public

---------

Co-authored-by: Craig Long <craig.long@autodesk.com>
  • Loading branch information
saintentropy and saintentropy committed Apr 24, 2023
1 parent 093f5a5 commit 5f7f7ea
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions src/DynamoCore/Extensions/LinterExtensionBase.cs
Expand Up @@ -15,7 +15,7 @@ namespace Dynamo.Extensions
/// <summary>
/// Base class for all LinterExtensions
/// </summary>
internal abstract class LinterExtensionBase : IExtension
public abstract class LinterExtensionBase : IExtension
{
private const string NODE_ADDED_PROPERTY = "NodeAdded";
private const string NODE_REMOVED_PROPERTY = "NodeRemoved";
Expand All @@ -26,9 +26,12 @@ internal abstract class LinterExtensionBase : IExtension
private LinterManager linterManager;
private HomeWorkspaceModel currentWorkspace;

internal ReadyParams ReadyParamsRef { get; set; }
public ReadyParams ReadyParamsRef { get; private set; }

internal bool IsActive => this.linterManager?.IsExtensionActive(UniqueId) ?? false;
/// <summary>
/// Is this linter currently active for the active workspace.
/// </summary>
public bool IsActive => this.linterManager?.IsExtensionActive(UniqueId) ?? false;

internal bool SetupComplete { get; set; } = false;

Expand Down Expand Up @@ -103,7 +106,7 @@ internal void Activate(bool linkToWorkspace = true)
internal void Deactivate()
{
//Nothing to deactivate if we have not setup everything properly
//yet or we alreay deativated this linter extension
//yet or we already deactivated this linter extension
if (!SetupComplete)
{
return;
Expand Down

0 comments on commit 5f7f7ea

Please sign in to comment.