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

OnVersioned.java:41-50: Remove VersionsMojo. It is not... #2381

Closed
0pdd opened this issue Aug 11, 2023 · 2 comments · Fixed by #2386
Closed

OnVersioned.java:41-50: Remove VersionsMojo. It is not... #2381

0pdd opened this issue Aug 11, 2023 · 2 comments · Fixed by #2386
Assignees
Milestone

Comments

@0pdd
Copy link

0pdd commented Aug 11, 2023

The puzzle 2376-3d36c5d7 from #2376 has to be resolved:

* @todo #2376:90min Remove VersionsMojo.
* It is not used anymore. Remove it and all its dependencies from all the places.
* We need to apply {@link OnVersioned} in {@link org.eolang.maven.DiscoverMojo}
* and replace all the tests from VersionsMojoTest to DiscoverMojoTest.
* Also we need to enable the next tests:
* - {@link org.eolang.maven.ProbeMojoTest#findsProbesWithVersionsInDifferentObjectionaries()}
* - {@link org.eolang.maven.PullMojoTest#pullsProbedVersionedObjectsFromDifferentObjectionaries()}
* - {@link org.eolang.maven.DiscoverMojoTest#discoversWithSeveralObjectsWithDifferentVersions()}
* - {@link org.eolang.maven.DiscoverMojoTest#discoversWithVersions()}
* Don't forget to remove that puzzle after all.

The puzzle was created by rultor on 11-Aug-23.

Estimate: 90 minutes, role: DEV.

If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@volodya-lombrozo volodya-lombrozo self-assigned this Aug 11, 2023
@Graur Graur added this to the Urgent milestone Aug 14, 2023
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Aug 14, 2023
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Aug 14, 2023
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Aug 14, 2023
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Aug 14, 2023
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Aug 14, 2023
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Aug 14, 2023
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Aug 14, 2023
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Aug 14, 2023
maxonfjvipon added a commit to maxonfjvipon/eo that referenced this issue Aug 14, 2023
@0pdd
Copy link
Author

0pdd commented Aug 14, 2023

@0pdd the puzzle #2389 is still not solved.

@0pdd
Copy link
Author

0pdd commented Aug 15, 2023

@0pdd the only puzzle #2389 is solved here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants