Would be good to refactor NonReversibleValidationException into exception package

[ignite1771]

Expected Behavior

All exception classes are categorized into the exception package.

Current Behavior

copybara/java/com/google/copybara/NonReversibleValidationException.java

Lines 17 to 36 in abec1ba

	package com.google.copybara;
	import net.starlark.java.eval.EvalException;
	/**
	* Exception thrown when a {@link Transformation} is not reversible but the configuration asked for
	* the reverse.
	*
	* TODO(malcon): Move to the exception package
	*/
	public class NonReversibleValidationException extends EvalException {
	public NonReversibleValidationException(String message) {
	super(message);
	}
	public NonReversibleValidationException(String message, Throwable cause) {
	super(message, cause);
	}
	}

There is a TODO in the comment of the file implies that this class should be also categorized into exception package.

Possible solution

I would like to send a PR to this, for refactoring the class and solving the affected bazel build dependency configuration.
By doing so, the source code would have better modularity.


Any pointers would be greatly appreciated.
Thanks!

[mikelalcon]

Sure, that would be great. My bet is that I didn't do it because there was a circular dependency, so it might be a bit tricky.

[ignite1771]

Got it. I will try to work on it.

[sheltshamu]

what's the update on this issues?

[ignite1771]

@sheltshamu Already merged in commit d052311 so will be closed.