Skip to content
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

feat: add log settings to project settings #796

Closed
wants to merge 5 commits into from

Conversation

Hertzole
Copy link
Contributor

This PR adds log settings to the Unity project window and also greatly improving saving and loading.

It now uses a scriptable object saved in the ProjectSettings/com.miragenet.mirage folder, so now all users will have the same log settings.
This scriptable object is also moved into the assets folder on build and then removed afterward so it can be used at runtime.

@Hertzole
Copy link
Contributor Author

Forgot to mention, this fixes #683 and #742

@James-Frowen
Copy link
Member

James-Frowen commented Apr 30, 2021

Looks promising, I want to take a few days to go through each part to make sure they all work.

I think this also fixes #791

@github-actions github-actions bot added the has conflicts Pull Request has merge conflicts label May 4, 2021
@James-Frowen James-Frowen removed the has conflicts Pull Request has merge conflicts label May 17, 2021
@James-Frowen
Copy link
Member

James-Frowen commented May 18, 2021

I'm going through this PR now. I want to make sure I understand what is going on before I merge it, and I'm probably going to try and simplify a few of the parts.

I'm also added headers for the namespaces
image

I'm hoping to get this all merged by the end of this week

Edit: I'm also looking into a "Find All" button, we might be able to use reflection to find all class that have a static readonly ILogger logger field.

@github-actions github-actions bot added the has conflicts Pull Request has merge conflicts label May 23, 2021
@phodoval
Copy link
Contributor

This has been merged in as #951.

@phodoval phodoval closed this Oct 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has conflicts Pull Request has merge conflicts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Log level arn't applied for test mode Log level are encumbersome to set LogSettings is broken
3 participants