Skip to content

Commit

Permalink
rem - brk|doc - Removed SettingVariable command flag
Browse files Browse the repository at this point in the history
---

We have removed SettingVariable because -set=varname already exists. The mod authors will have to use the AcceptsSet parameter from the constructor of CommandArgumentInfo.

---

Type: rem
Breaking: True
Doc Required: True
Part: 1/1
  • Loading branch information
AptiviCEO committed Feb 9, 2024
1 parent 2475b43 commit 818526c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 8 deletions.
2 changes: 1 addition & 1 deletion public/Nitrocid/Nitrocid.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@
<NitrocidModAPIVersionMajor>3.0.25</NitrocidModAPIVersionMajor>

<!-- Increment NitrocidModAPIVersionChangeset every time there is a breaking change or an API addition in the N-KS API. -->
<NitrocidModAPIVersionChangeset>415</NitrocidModAPIVersionChangeset>
<NitrocidModAPIVersionChangeset>416</NitrocidModAPIVersionChangeset>

<!-- To be installed to the file version -->
<NitrocidModAPIVersion>$(NitrocidModAPIVersionMajor).$(NitrocidModAPIVersionChangeset)</NitrocidModAPIVersion>
Expand Down
9 changes: 2 additions & 7 deletions public/Nitrocid/Shell/ShellBase/Commands/CommandFlags.cs
Original file line number Diff line number Diff line change
Expand Up @@ -43,17 +43,12 @@ public enum CommandFlags
/// </summary>
Obsolete = 4,
/// <summary>
/// The command is setting a variable.
/// </summary>
[Obsolete("-set=varname already exists. Use the AcceptsSet parameter from the CommandArgumentInfo constructor instead of this flag.")]
SettingVariable = 8,
/// <summary>
/// Redirection is supported, meaning that all the output to the commands can be redirected to a file.
/// </summary>
RedirectionSupported = 16,
RedirectionSupported = 8,
/// <summary>
/// This command is wrappable to pages.
/// </summary>
Wrappable = 32,
Wrappable = 16,
}
}

0 comments on commit 818526c

Please sign in to comment.