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

Minor fixes for CoordinateTransformTester, as per #41 and #42 #47

Merged
merged 1 commit into from
Aug 26, 2019
Merged

Minor fixes for CoordinateTransformTester, as per #41 and #42 #47

merged 1 commit into from
Aug 26, 2019

Conversation

Neutius
Copy link
Contributor

@Neutius Neutius commented Aug 25, 2019

As per paulushub on f8a9eaa:

  1. Removing @test: This class is not directly involved in the test process, the addition of the attribute produces several initialization errors.
  2. Removing unused fields p, p2 and the junit package.

Final test results:
Before Updates: Runs: 72/72 (2 skipped) Errors: 28 Failures: 1
After Updates: Runs: 44/44 (2 skipped) Errors: 0 Failures: 1

Signed-off-by: Wambertus sfcgijs@gmail.com

Resolves #41 and resolves #42

1. Removing @test: This class is not directly involved in the test process, the addition of the attribute produces several initialization errors.
2. Removing unused fields p, p2 and the junit package.

Final test results:
Before Updates: Runs: 72/72 (2 skipped)            Errors: 28             Failures: 1
After Updates: Runs: 44/44 (2 skipped)               Errors: 0               Failures: 1

Signed-off-by: Wambertus <sfcgijs@gmail.com>
@Neutius
Copy link
Contributor Author

Neutius commented Aug 25, 2019

I'm having some issues with the legal agreement, due to using both my work e-mail and my private e-mail. Will fix tomorrow.

Edit: fixed. Made a second Eclipse account with my private e-mail.

@pomadchin
Copy link
Member

Thanks for doing it @Neutius!

@echeipesh
Copy link
Contributor

ECA is now good (We should really add the automatic checker to GitHub here).
Thank you for another contribution.

@echeipesh echeipesh merged commit 73d0095 into locationtech:master Aug 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CoordinateTransformTester
3 participants