This repository was archived by the owner on Sep 13, 2022. It is now read-only.
Merged
Conversation
- reorder final static keywords
- reorder public abstract, private static, public static keywords
- provide parametrized type when necessary (there are still a few)
- disable warnings in PC/SC (uses reflexivity)
- create ProxyElement - delete Nameable.java and Configurable.java - move AbstractSeProxyComponent in plugin package, make private-package
- make the increase/decrease anticipated responses correct
- now starting at 0 as in previous versions
Feat/keyp12 244 sonarqube
# Conflicts: # java/component/keyple-calypso/src/main/java/org/eclipse/keyple/calypso/transaction/SamResourceManagerDefault.java # java/component/keyple-calypso/src/main/java/org/eclipse/keyple/calypso/transaction/SamResourceManagerFactory.java # java/component/keyple-calypso/src/main/java/org/eclipse/keyple/calypso/transaction/SamResourceManagerPool.java
Signed-off-by: Meddy Menzikoff <m.menzikoff@oneclickflare.fr>
0.8.1 merge in develop
…eturns_plugin Refact: SeProxyService.registerPlugin now returns registered plugin.
… command) Signed-off-by: Andrei Cristea <andrei.cristea019@gmail.com>
…_calypso_po feat: KEYP-12: add method "fillContent" to CalypsoPo (for WriteRecord…
- remove IsoAid inner class from SeSelector.AidSelector - apply the builder pattern to all selectors (SeSelector, PoSelector, SamSelector, SamIdentifier) - update unit tests and examples
- changes to builder pattern with inheritance - update javadoc
- remove IsoAid inner class from SeSelector.AidSelector - apply the builder pattern to all selectors (SeSelector, PoSelector, SamSelector, SamIdentifier) - update unit tests and examples
* KeypleReaderException is now thrown by all class implementing AbstractPlugin * Removed KeypleRuntimeException from AbstractPlugin constructor * AbstractPluginFactory us now an interface (to be renamed!) * All exceptions in getPluginInstances() implementations are now wrapped into KeyplePluginInstanciationException. Signed-off-by: Meddy Menzikoff <m.menzikoff@oneclickflare.fr>
Signed-off-by: Meddy Menzikoff <m.menzikoff@oneclickflare.fr>
- remove prefixes SeSelector and PoSelector whenever possible
Signed-off-by: Meddy Menzikoff <m.menzikoff@oneclickflare.fr>
- the "Google" format replaces the previous one - the format of the header file is updated as well
- all source files are updated: format and header
Signed-off-by: Meddy Menzikoff <m.menzikoff@oneclickflare.fr>
…setting Refact/keyp 246 spotless format setting
…ing_for_alpha2 Fix: Spotless alert
…ectoryHeader class
…rrections Refact/keyp 228 236 299 minor corrections
* refactor: KEYP-323 move reader classes to a dedicated reader package
#394) * Refact: Changed prototype of parameters and methods exposing internal CoucurrentHashMaps Signed-off-by: Meddy Menzikoff <m.menzikoff@oneclickflare.fr> * Refact: Updating tests Signed-off-by: Meddy Menzikoff <m.menzikoff@oneclickflare.fr> * Refact: Cleaning Signed-off-by: Meddy Menzikoff <m.menzikoff@oneclickflare.fr>
* fix: KEYP-244 reduce cognitive complexity * fix: KEYP-244 fix keyple-core issues * fix: KEYP-244 fix keyple-core tests issues * fix: KEYP-244 fix keyple-calypso issues * fix: KEYP-244 fix keyple-calypso tests issues * fix: KEYP-244 fix keyple-plugin issues * fix: KEYP-244 remove unnecessary overloads of toString * fix: KEYP-244 another batch of fixes for issues raised by SonarQube * fix: KEYP-244 remove serialVersionUID fields
* fix: KEYP-322 remove TransmissionMode from AbstractMatchingSe - retrieve the transmission mode from the reader instead of CalypsoPo in PoTransaction * fix: KEYP-322 update the PC/SC plugin - move settings strings in separate classes * fix: KEYP-322 update the PC/SC plugin - change transmission mode management - add the possibility to find the transmission from the reader name (plugin/findTransmissionMode) - correct reader/getTransmissionMode * fix: KEYP-244 update examples * fix: KEYP-322 spotless * fix: KEYP-322 update examples * fix: KEYP-322 spotless * fix: KEYP-322 spotless configuration (examples) * fix: KEYP-322 update examples (Android) * fix: KEYP-322 make SeProtocol truly optional in PoSelector and SamSelector - there is no more SeProtocol defined by default * fix: KEYP-322 update transmission mode management in CalypsoSam related classes * fix: KEYP-322 refinement of PC/SC classes * fix: KEYP-322 various update in PC/SC plugin
- suppression and simplification as much as possible of the common elements - simplification of the readers' parameterization - various corrections and optimizations
…tings refact: KEYP-328 deep reorganisation of examples
refact: KEYP-331 Update example to illustrate sync/async updates
* Fix raw readme for version 0.9 * Remove old Core & Calypso API guides, now hosted on keyple.org * readme typo
* Adding Release to Jenkinsfile Signed-off-by: Brice Ruppen <brice.ruppen@armotic.fr> * Release keyple-java 0.9.0 Signed-off-by: Brice Ruppen <brice.ruppen@armotic.fr>
Signed-off-by: Brice Ruppen <brice.ruppen@armotic.fr>
fix: release does not need to create new commit
jeanpierrefortune
approved these changes
Sep 10, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.