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

expand diff.groovy to support sevntu #340

Open
rnveach opened this issue Dec 7, 2018 · 1 comment
Open

expand diff.groovy to support sevntu #340

rnveach opened this issue Dec 7, 2018 · 1 comment

Comments

@rnveach
Copy link
Member

rnveach commented Dec 7, 2018

Discussed at sevntu-checkstyle/sevntu.checkstyle#720 (comment) , diff.groovy doesn't support sevntu regression.

Program needs to examine given repo and understand if it is checkstyle or sevntu.
Install needs to be updated to run a different command for sevntu repo.
Program needs to be updated to swap sevntu versions while keeping checkstyle constant for sevntu repo.

Documentation has to be updated for new sevntu options and to describe how to run regression for sevntu.

@rnveach
Copy link
Member Author

rnveach commented Jun 7, 2021

When implementing, we should try to make this feature as repo agnostic as possible. Then if we ever have a special repo like in GSoC, we can be sure we can regress on it without needing to merge it in first or do anything special.

We can just use sevntu's repo as a base which requires a property for checkstyle version, directory to run command, and any special command needed in packaging.

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

No branches or pull requests

1 participant