Skip to content

Conversation

@thboileau
Copy link
Contributor

The aim

Mavenify the projects by dispatching the existing test cases into the convenient module.

Check-list

  • PR size
    • Under 300 lines ✅
    • Can't be split without complicating the process even more
  • Tests
    • Added
    • Not applicable (why?)
  • Doc
    • Added
    • Not applicable
  • Reviewer
    • Asked for a review
    • Added label DO NOT REVIEW

@thboileau thboileau merged commit c84baac into 2.6 Feb 2, 2025
1 check passed
@thboileau thboileau deleted the 2.6__1446_dispatch-tests-maven branch February 2, 2025 13:28

/** The server nonce count. */
private volatile int serverNounceCount;
private volatile int serverNonceCount;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the word nonce was mispelled. I've deprecated the accessors and add new ones.

@@ -0,0 +1,36 @@
/**
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot to add the tests for the GWT edition

/**
* Copyright 2005-2024 Qlik
*
* <p>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we'll have to take care of code formatting...

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