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.

Sign up for GitHub

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

Jump to bottom

Update JSON Lib #6

Closed
HackingPheasant opened this issue Jul 16, 2021 · 0 comments
Closed

Update JSON Lib #6

HackingPheasant opened this issue Jul 16, 2021 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@HackingPheasant
Copy link
Owner

Since nlohmann/json#2762 has been merged it may be best to move to that git tag (or release if it gets made before I implement the change) to silence the 3rd Party Library warnings showing up in our code.
@HackingPheasant HackingPheasant added enhancement New feature or request good first issue Good for newcomers labels Jul 16, 2021
HackingPheasant added a commit that referenced this issue Nov 15, 2021
@HackingPheasant
everything: Massive (overdue) cleanup
6493881 
- Update 3rd party libraries to the newest version (Closes #6)
- Removed Mangadex code since I will need to redo all of it anyways
- Move Windows NOMINMAX define out of MSVC compiler to an if(Windows)
  conditional (Closes #7)
- Move MSVC /permissive- flag outside of the if(ENABLE_WARNINGS)
  conditional
  (Closes #7)
- Merge Clang and AppleClang flags (again)
- Clean-up top level CMakeLists.txt by moving the changes to appropriate
  .cmake files
- Drop generator from CMakePresets.json now that its not required by
  default.
- Update .gitignore
- Restructure how I handle handle source files. The time to do this sort
  of thing is now, figuring out what works best for me and the project is
  when its infancy and I am the sole contributor.

We do include the beginning skeleton of the new window.[cpp/h] files
since it was easier then more mucking about in the cmake files. ;)
HackingPheasant added a commit that referenced this issue Sep 2, 2024
@HackingPheasant
everything: Massive (overdue) cleanup
88a5621 
- Update 3rd party libraries to the newest version (Closes #6)
- Removed Mangadex code since I will need to redo all of it anyways
- Move Windows NOMINMAX define out of MSVC compiler to an if(Windows)
  conditional (Closes #7)
- Move MSVC /permissive- flag outside of the if(ENABLE_WARNINGS)
  conditional
  (Closes #7)
- Merge Clang and AppleClang flags (again)
- Clean-up top level CMakeLists.txt by moving the changes to appropriate
  .cmake files
- Drop generator from CMakePresets.json now that its not required by
  default.
- Update .gitignore
- Restructure how I handle handle source files. The time to do this sort
  of thing is now, figuring out what works best for me and the project is
  when its infancy and I am the sole contributor.

We do include the beginning skeleton of the new window.[cpp/h] files
since it was easier then more mucking about in the cmake files. ;)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
Inital 1.0 Release
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@HackingPheasant