commit | d5d3bd75d1be5691b1d8dab5a5592812a2a5da76 | [log] [tgz] |
---|---|---|
author | Martin Wittlinger <wittlinger.martin@gmail.com> | Wed May 17 14:38:46 2023 +0200 |
committer | GitHub <noreply@github.com> | Wed May 17 08:38:46 2023 -0400 |
tree | 134476ab2bfb8d6711f82484652e409cb9fcfe24 | |
parent | 57edc03162557d052deec762d6b87f1a4c24d653 [diff] |
refactor: refactor bad smell UnnecessaryToStringCall (#186) * Refactor bad smells: - UnnecessaryToStringCall The toString() method is not needed in cases the underlying method handles the conversion. Also calling toString() on a String is redundant. Removing them simplifies the code.
You have found a bug or you have an idea for a cool new feature? Contributing code is a great way to give something back to the open source community. Before you dig right into the code, there are a few guidelines that we need contributors to follow so that we can have a chance of keeping on top of things.
We accept Pull Requests via GitHub. The developer mailing list is the main channel of communication for contributors.
There are some guidelines which will make applying PRs easier for us:
git diff --check
before committing.[MSHADE-XXX] - Subject of the JIRA Ticket Optional supplemental description.
mvn -Prun-its verify
to assure nothing else was accidentally broken.If you plan to contribute on a regular basis, please consider filing a contributor license agreement.
For changes of a trivial nature to comments and documentation, it is not always necessary to create a new ticket in JIRA. In this case, it is appropriate to start the first line of a commit with ‘(doc)’ instead of a ticket number.
If your machine is sufficiently powerful and you want to parallelise the IT execution to validate the build before a PR you can set the concurrency in MAVEN_OPTS
:
MAVEN_OPTS=-Dinvoker.parallelThreads=2 mvn verify -Prun-its
You can also run a single IT test using:
mvn verify -Prun-its -Dinvoker.test=myitproject