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

[MNG-6869] New flag to verify Maven installation status #995

Open
wants to merge 44 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
44 commits
Select commit Hold shift click to select a range
904caa4
[MNG-6869] Initial idea
Giovds Dec 7, 2022
2dab934
[MNG-6869] Further brainstorming
mthmulders Dec 7, 2022
1082a0d
[MNG-6869] Refactor local variable name
Giovds Dec 7, 2022
8d46d92
[MNG-6969] WIP commit
Giovds Dec 22, 2022
9aaf62f
[MNG-6869] Correct way to find local repo path
mthmulders Dec 22, 2022
57c7020
[MNG-6869] Use java.nio instead of java.io
Giovds Dec 22, 2022
55f35fd
[MNG-6869] Use java.nio instead of java.io
mthmulders Dec 22, 2022
fdcce30
[MNG-6869] Implement verification of artifact resolution (WIP)
mthmulders Dec 22, 2022
e4ec517
[MNG-6869] Test artifact resolution with temporary local repository
mthmulders Jan 12, 2023
77526f5
[MNG-6869] Setup checks for remote connections
Giovds Jan 12, 2023
fbbcc20
[MNG-6869] Formatting
mthmulders Jan 12, 2023
ba9354a
[MNG-6869] Refactor: rename method
mthmulders Jan 12, 2023
85b0907
[MNG-6869] Formatting
mthmulders Jan 16, 2023
12a6b4b
[MNG-6869] Verify connection to remote repositories
mthmulders Jan 26, 2023
140a63c
[MNG-6869] Reformat with Spotless
mthmulders Jan 26, 2023
7afb18e
[MNG-6869] Add todo to fix local repository issue
Giovds Jan 26, 2023
ca4cb33
[MNG-6869] Narrow declaration of thrown exception
mthmulders Jan 26, 2023
0764fd0
[MNG-6869] Refine detection of possible connection issues
mthmulders Jan 26, 2023
30c9649
[MNG-6869] Use local repository correctly
Giovds Feb 9, 2023
d134358
[MNG-6869] Clean up temp file after usage
Giovds Feb 9, 2023
6ba97e6
[MNG-6869] Include local repository path in issue text
mthmulders Feb 9, 2023
79338ae
[MNG-6869] Mention local repo check has been completed
mthmulders Feb 9, 2023
b470aca
[MNG-6869] Output detected issues prefixed with a number
mthmulders Feb 9, 2023
8629a8a
[MNG-6869] Properly close streeam
mthmulders Feb 9, 2023
7fcc380
[MNG-6869] Make logger a static constant
mthmulders Feb 9, 2023
2b01442
[MNG-6869] Make it explicit we're populating instance variables
mthmulders Feb 9, 2023
95c52f2
[MNG-6869] Reorder instance variables
mthmulders Feb 9, 2023
d0f3b29
[MNG-6869] Mark method arguments as final
mthmulders Feb 9, 2023
58e2f46
[MNG-6869] Store temporary local repo as Path
mthmulders Feb 9, 2023
f67f7c3
[MNG-6869] Rename logger to LOGGER
mthmulders Feb 9, 2023
27a7e0a
[MNG-6869] Checkstyle: redundant modifier
mthmulders Feb 9, 2023
d2416a2
[MNG-6869] Minor review points
mthmulders Feb 9, 2023
2a9ce5e
[MNG-6869] Remove commented code
Giovds Feb 23, 2023
ed191ad
[MNG-6869] Use exception classification of Transporter
Giovds Feb 23, 2023
6adddbd
[MNG-6869] Make formatException signature non-optional
Giovds Feb 23, 2023
9443a2a
[MNG-6869] Remove unnecessary comment
Giovds Feb 23, 2023
41177b3
[MNG-6869] Remove unused DI fields
Giovds Feb 23, 2023
fc2e7f7
[MNG-6869] Make no assumptions about the existence of Maven Central
mthmulders Feb 23, 2023
2114cda
[MNG-6869] Simplify
mthmulders Feb 23, 2023
853819c
[MNG-6869] Prevent possible ClassCastException
mthmulders Feb 23, 2023
f39aec2
[MNG-6869] Revert unrelated change
mthmulders May 10, 2023
6c4348d
Merge branch 'master' into mng-6869-new-flag-to-verify-maven-status
Giovds Apr 26, 2024
eaf2421
[MNG-6869] Make it work with recent changes
Giovds Apr 26, 2024
63006df
[MNG-6869] Make use of try with resources
Giovds Apr 26, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[MNG-6869] Rename logger to LOGGER
  • Loading branch information
mthmulders committed Feb 23, 2023
commit f67f7c3e0329316b16b4d6e51632c26f55cd4fda
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 61,7 @@
import org.slf4j.LoggerFactory;

public class MavenStatusCommand {
private static final Logger logger = LoggerFactory.getLogger(MavenStatusCommand.class);
private static final Logger LOGGER = LoggerFactory.getLogger(MavenStatusCommand.class);

/**
* In order to verify artifacts can be downloaded from the remote repositories we want to resolve an actual
Expand Down Expand Up @@ -122,7 122,7 @@ private void cleanupTempFiles() {
.map(Path::toFile)
.forEach(File::delete);
} catch (IOException ioe) {
logger.debug("Failed to delete temporary local repository", ioe);
LOGGER.debug("Failed to delete temporary local repository", ioe);
}
}
}
Expand All @@ -133,8 133,8 @@ private void setTemporaryLocalRepositoryPathOnRequest(final MavenExecutionReques
request.setLocalRepositoryPath(tempLocalRepository.toString());
request.setLocalRepository(repositorySystem.createLocalRepository(request, tempLocalRepository.toFile()));
} catch (Exception ex) {
logger.debug("Could not create temporary local repository", ex);
logger.warn("Artifact resolution test is less accurate as it may use earlier resolution results.");
LOGGER.debug("Could not create temporary local repository", ex);
LOGGER.warn("Artifact resolution test is less accurate as it may use earlier resolution results.");
}
}

Expand Down Expand Up @@ -170,14 170,14 @@ private List<String> verifyArtifactResolution(final MavenExecutionRequest mavenE
resolverResult
.getArtifacts()
.forEach((key, value) ->
logger.debug("Successfully resolved {} to {}", key.toString(), value.toString()));
LOGGER.debug("Successfully resolved {} to {}", key.toString(), value.toString()));
mthmulders marked this conversation as resolved.
Show resolved Hide resolved

return Collections.emptyList();
} catch (ArtifactResolverException are) {
return extractIssuesFromArtifactResolverException(are);
} finally {
sessionScope.exit();
logger.info("Artifact resolution check completed");
LOGGER.info("Artifact resolution check completed");
}
}

Expand Down Expand Up @@ -211,7 211,7 @@ private List<String> verifyLocalRepository(final Path localRepositoryPath) {
issues.add(String.format("No write permissions on local repository %s.", localRepositoryPath));
}

mthmulders marked this conversation as resolved.
Show resolved Hide resolved
logger.info("Local repository setup check completed");
LOGGER.info("Local repository setup check completed");
return issues;
}
}