-
-
Notifications
You must be signed in to change notification settings - Fork 126
Implement C++ Mod Config #950
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Sora-yx
wants to merge
28
commits into
Reloaded-Project:master
Choose a base branch
from
Sora-yx:feature/cpp-mod-configuration
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
4c8e61c
Implement C++ Mod Config.
Sora-yx 8bbcc06
Fix typo in the doc (ended dropping crossplatform)
Sora-yx 7fb4acf
ModConfig: Fix missing 32 bits mod template reference.
Sora-yx 00d437e
Syntax moment
Sora-yx a1658e0
Handle null case for when user config directory is null (+ rework mig…
Sora-yx a423fff
Validate filename helper + add missing "Values" for enum
Sora-yx e924738
Inlined enum stuff through helper functions
Sora-yx 7308d68
Invoke and Migrate improvement stuff.
Sora-yx 9671e70
Reloaded Mod Config header: Add mutex for thread safe + API struct
Sora-yx 8db149d
Documentation fixes: formatting adjust + rework CMake to use env vari…
Sora-yx ef58379
wrapper to the loader API
Sora-yx 3b0573e
Changed: Improve/shorten NativeMods docs
Sewer56 da26497
Changed: Shorten native mod template README
Sewer56 9055250
Changed: Remove unused usings from native config tests
Sewer56 98c866e
Changed: Polish native config docs and clear new-code warnings
Sewer56 756a7a8
Changed: Group native mod config types under Native subnamespace
Sewer56 1f8bd02
Changed: Split native mod config schema models into Schema subnamespace
Sewer56 405b14c
Changed: Link native schema docs to C# interface types
Sewer56 14c1c83
Update: Doc cleanup of ConfigurableBase.cs
Sewer56 1dfb6cb
Style: ModConfigSchema added newline
Sewer56 a15949f
Style: Fix initializer alignment in picker schema parsers
Sewer56 eadcbad
Fixed: Remove duplicate blank line in ConfigureModCommand
Sewer56 7b9e43e
Changed: Link plain doc mentions to code members
Sewer56 0fa2850
Changed: Dedupe native and managed configurator setup
Sewer56 e799103
Changed: Add Arrange/Act/Assert sections to native mod tests
Sewer56 cc6ff21
Changed: Removed redundant comment
Sewer56 0aa172d
Native Mod Config: swap to Enum + add missing json exception + valida…
Sora-yx 86c1e4d
Updated documentation to add enum property and InitialFolderPath
Sora-yx File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs hyperlink to MSDN.
And separate bullet; to avoid complexity making things hard for user to read.