Skip to content

Conversation

Copilot
Copy link

@Copilot Copilot AI commented Aug 25, 2025

  • Add missing static imports as recommended by style guide
    • Add static imports for commonly used utility methods (Arrays, Collections, Objects, Locale, Level)
    • Add static imports for test assertion methods in test files
  • Apply final keyword usage rules from style guide
    • Make public non-internal classes final where possible
    • Ensure fields are final where possible
  • Verify class organization follows recommended order
    • Static fields (final before non-final)
    • Instance fields (final before non-final)
    • Constructors
    • Methods
    • Nested classes
  • Run tests to ensure changes don't break functionality
  • Apply spotless formatting to ensure code style compliance

All style guide requirements have been successfully implemented with minimal changes. The module now follows:

  • Static imports for commonly used utility methods and constants
  • Proper final keyword usage on public classes
  • Test files use static imports for assertion methods
  • All tests pass and code formatting is compliant

Fixes #393.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

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.

Update jmx-scraper module to follow style guide
2 participants