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

Improve code quality #35

Closed
wants to merge 26 commits into from
Closed

Improve code quality #35

wants to merge 26 commits into from

Conversation

fruzitent
Copy link

TODO

Application

  • Add cpplint to the build system
  • Reduce tests compile time
  • Simplify code example to basic GUI window
  • Use code-formatting tools

Devcontainers

  • Build tools from source to avoid weird depency issues
  • Figure out how to distribute pre-built container
  • Find a way to make VSCode respect pull_policy
  • Extract VSCode settings into own files
  • Refactor Dockerfile

Documentation

  • Update building commands

Workflows

  • Update building commands

@lefticus
Copy link
Member

I'm sorry but this PR is FAR too large to be successfully reviewed and merged, and incorporates a lot of changes and many of them are stylistic opinions, such as formatting of CMake scripts and changes to markdown - vs *.

Please make smaller PRs if you would like them reviewed for inclusion

@fruzitent fruzitent closed this May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants