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

Cleanup Project #15

Merged
merged 35 commits into from
Oct 22, 2024
Merged

Cleanup Project #15

merged 35 commits into from
Oct 22, 2024

Conversation

celestialorb
Copy link
Owner

This PR cleans up large parts of this project by correcting linter findings, formatting files, laying down some IDE settings, and organizing the repository.

This PR also adds two new features:

  • Introduces a mod setting to control how often metrics are written out to the output file.
  • Adds localization for mod settings.

@celestialorb
removing extra whitespace
07e1354
@celestialorb
making method static
25c8526
@celestialorb
bumping patch version
1a17821
@celestialorb
simplifying value conversion for metric
cb5737b
@celestialorb
updating language
106fd10
@celestialorb
removing C901 finding
3f67a52
@celestialorb
using more descriptive name
3a88ab9
@celestialorb
selecting all linter issues
8b1f8eb
@celestialorb
shortening longer lines
1db31f0
@celestialorb
using success logging
a3d447b
@celestialorb
increasing acceptable line length
f7a27ce
@celestialorb
adding VSCode settings
0e00fba
@celestialorb
sorting imports
36e019f
@celestialorb
using exception logging over errors
16a8c9d
@celestialorb
simplify foreach loop
3594522
@celestialorb
using success logs over green info logs
e3a0fa0
@celestialorb
forcing boolean parameters to keyword arguments
a2a4720
@celestialorb
fixing import
e34e98e
@celestialorb
formatting on save in VSCode
2a66902
@celestialorb
adding Ruff as formatter for Python
79a0af6
@celestialorb
adding newline from formatter
f8d1cbe
@celestialorb
introducing lock for metrics data
338de6d
@celestialorb
cleaning up loading of metric data
822962e
@celestialorb
adding log Lua global
d784e4e
@celestialorb
fixing CLI
8d30d00
@celestialorb
using lowercase for log messages
71b7f47
@celestialorb
include missing threading import
a54b9e2
@celestialorb
reorganize metric collection
8fdd719
@celestialorb
moving mod components to mod subfolder
d3cc9b2
@celestialorb
adding localization
c73d1af
@celestialorb celestialorb self-assigned this Oct 22, 2024
@celestialorb celestialorb marked this pull request as ready for review October 22, 2024 20:38
@celestialorb celestialorb merged commit 3e6abb6 into main Oct 22, 2024
1 check passed
@celestialorb celestialorb deleted the improvement/cleanup/productionize branch October 22, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@celestialorb celestialorb

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@celestialorb