Tag List Report
The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.
| Tag Class | Total number of occurrences | Tag strings used by tag class |
|---|---|---|
| @todo | 12 | @todo |
| TODO | 20 | TODO |
Each tag is detailed below:
@todo
Number of occurrences found in the code: 12
| org.apache.maven.shared.release.phase.CheckDependencySnapshotsPhase | Line |
|---|---|
| plugins with no version will be resolved to RELEASE which is not a snapshot, but remains unresolved to this point. This is a potential hole in the check, and should be revisited after the release pom writing is done and resolving versions to verify whether it is. | 53 |
| plugins injected by the lifecycle are not tested here. They will be injected with a RELEASE version so are covered under the above point. | 54 |
| check dependencyManagement | 143 |
| check pluginManagement | 148 |
| check profiles | 158 |
| org.apache.maven.shared.release.phase.IsScmFileSetEquals | Line |
| add an equals() method | 31 |
| org.apache.maven.shared.release.phase.IsScmTagEquals | Line |
| add an equals() method | 29 |
| org.apache.maven.shared.release.phase.MapVersionsPhase | Line |
| validate default version, maybe with DefaultArtifactVersion | 240 |
| validate next version, maybe with DefaultArtifactVersion | 302 |
| org.apache.maven.shared.release.phase.ScmCheckModificationsPhase | Line |
| proper construction of filenames, especially release properties | 77 |
| org.apache.maven.shared.release.scm.ScmTranslator | Line |
| move this API into SCM? | 26 |
| org.apache.maven.shared.release.versions.VersionInfo | Line |
| this has an overlap with ArtifactVersion in maven-artifact - would be good to migrate this there. | 23 |
TODO
Number of occurrences found in the code: 20
| org.apache.maven.shared.release.DefaultReleaseManager | Line |
|---|---|
| rollback? | 254 |
| org.apache.maven.shared.release.exec.InvokerMavenExecutor | Line |
| Configuring an invocation request from a command line could as well be part of the Invoker API | 177 |
| setQuiet() currently not supported by InvocationRequest | 220 |
| org.apache.maven.shared.release.phase.AbstractReleaseTestCase | Line |
| return void since this is redundant | 263 |
| org.apache.maven.shared.release.phase.AbstractRewritePomsPhase | Line |
| needs a better error message, what pom? what dependency? | 587 |
| org.apache.maven.shared.release.phase.CheckoutProjectFromScm | Line |
| sanity check that it is not . or .. or lower | 183 |
| org.apache.maven.shared.release.phase.GenerateReleasePomsPhase | Line |
| share with RewritePomsForReleasePhase.rewriteVersion | 338 |
| share with RewritePomsForReleasePhase.translateScm | 359 |
| org.apache.maven.shared.release.phase.GenerateReleasePomsPhaseTest | Line |
| MRELEASE-262 public void testRewriteInternalRangeDependency() throws Exception { List reactorProjects = createReactorProjects( "internal-snapshot-range-dependency" ); ReleaseDescriptor config = createMappedConfiguration( reactorProjects ); | 74 |
| org.apache.maven.shared.release.phase.RemoveScmTagPhase | Line |
| [!]: implement | 44 |
| org.apache.maven.shared.release.phase.RewritePomsForBranchPhase | Line |
| svn utils should take care of prepending this | 122 |
| org.apache.maven.shared.release.phase.RewritePomsForReleasePhase | Line |
| svn utils should take care of prepending this | 122 |
| org.apache.maven.shared.release.phase.ScmCheckModificationsPhase | Line |
| would be nice for SCM status command to do this for me. | 146 |
| org.apache.maven.shared.release.phase.ScmTagPhase | Line |
| want includes/excludes? | 112 |
| org.apache.maven.shared.release.scm.DefaultScmRepositoryConfigurator | Line |
| this is a bit dodgy - id is not host, but since we don't have a <host> field we make an assumption | 107 |
| org.apache.maven.shared.release.versions.DefaultVersionInfo | Line |
| this parser is better than DefaultArtifactVersion - replace it with this (but align naming) and then remove this from here. | 57 |
| hack because it didn't support "SNAPSHOT" | 128 |
| this is a workaround for a bug in DefaultArtifactVersion - fix there - 1.01 < 1.01.01 | 229 |
| this is a workaround for a bug in DefaultArtifactVersion - fix there - it should not consider case in comparing the qualifier NOTE: The combination of upper-casing and lower-casing is an approximation of String.equalsIgnoreCase() | 242 |
| org.apache.maven.shared.release.versions.DefaultVersionInfoTest | Line |
| bug in Artifact pattern checkGetSnapshotVersion( "20051112.134500-1", "SNAPSHOT" ); | 229 |