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

refactor: introduce commons module #251

Merged
merged 10 commits into from
Oct 16, 2024
Merged

Commits on Oct 15, 2024

  1. introduce the common module, abstract some literals

    Signed-off-by: Atanas Atanasov <[email protected]>
    ata-nas authored and jsync-swirlds committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    46969a5 View commit details
    Browse the repository at this point in the history
  2. abstract logic from existing utility classes

    Signed-off-by: Atanas Atanasov <[email protected]>
    ata-nas authored and jsync-swirlds committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    ea0d11b View commit details
    Browse the repository at this point in the history
  3. abstracting of more logic

    Signed-off-by: Atanas Atanasov <[email protected]>
    ata-nas authored and jsync-swirlds committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    bd7798f View commit details
    Browse the repository at this point in the history
  4. addressing comment

    Signed-off-by: Atanas Atanasov <[email protected]>
    ata-nas authored and jsync-swirlds committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    887e763 View commit details
    Browse the repository at this point in the history
  5. add todo

    Signed-off-by: Atanas Atanasov <[email protected]>
    ata-nas authored and jsync-swirlds committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    e1c9160 View commit details
    Browse the repository at this point in the history
  6. resolved todo

    Signed-off-by: Atanas Atanasov <[email protected]>
    ata-nas authored and jsync-swirlds committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    764bfc4 View commit details
    Browse the repository at this point in the history
  7. address some comments

    Signed-off-by: Atanas Atanasov <[email protected]>
    ata-nas authored and jsync-swirlds committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    6bbfba2 View commit details
    Browse the repository at this point in the history
  8. address more comments

    Signed-off-by: Atanas Atanasov <[email protected]>
    ata-nas authored and jsync-swirlds committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    b0f09e9 View commit details
    Browse the repository at this point in the history
  9. address pr comments based on new agreements

    Signed-off-by: Atanas Atanasov <[email protected]>
    ata-nas authored and jsync-swirlds committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    0d3475b View commit details
    Browse the repository at this point in the history

Commits on Oct 16, 2024

  1. Addressed review and other items

    * Fixed todo item by overloading method and moving default to local constant
    * Fixed some formatting issues
    * Fixed string utility to match more typical conventions
    * Modified "default" file permissions to not make everything executable
    * Renamed "abbreviated" classes to use full names
    * Renamed unsafe file methods to add "Unsafe" at the end
    * Documented why "unsafe" methods are not safe for production use
    * General cleanup
    
    Fixed a build headache
    * Modified spotless conventions to remove javadoc formatting for now
       * This was creating issues and conflicts between different tools; we'll need
         to add a proper _separate_ javadoc format once we can figure out how to
         do so within the ultra-strict spotless "write a new plugin" approach.
    
    Signed-off-by: Joseph Sinclair <[email protected]>
    jsync-swirlds committed Oct 16, 2024
    Configuration menu
    Copy the full SHA
    a1723d2 View commit details
    Browse the repository at this point in the history