sevntu-checkstyle / sevntu.checkstyle

Additional Checkstyle checks, that could be added as extension to EclipseCS plugin and maven-checkstyle-plugin, Sonar checkstyle plugin, extension for CheckStyle IDEA plugin.
http://sevntu-checkstyle.github.io/sevntu.checkstyle/
190 stars 147 forks source link

infra: removes majority of validation from travis and switch to GH #994

Closed rnveach closed 1 year ago

rnveach commented 1 year ago

Removes travis as it fails too much.

rnveach commented 1 year ago

https://github.com/sevntu-checkstyle/sevntu.checkstyle/actions/runs/4328888245/jobs/7559005360

Run ./.ci/validation.sh sonarqube SONAR_TOKEN is not set

@romani How can we fix this?

rnveach commented 1 year ago

run (eclipse-cs) java.io.FileNotFoundException: https://raw.githubusercontent.com/checkstyle/checkstyle/checkstyle-10.8.0/config/checkstyle_checks.xml

run (sonar-plugin) ResourceNotFoundException: Could not find resource 'https://raw.githubusercontent.com/checkstyle/checkstyle/checkstyle-10.8.0/config/checkstyle_checks.xml'.

run (checkstyle-regression) IllegalStateException: Can't read config: /home/runner/work/sevntu.checkstyle/sevntu.checkstyle/sevntu-checks/../.ci-temp/checkstyle/config/checkstyle_sevntu_checks.xml

All seem blocked by https://github.com/sevntu-checkstyle/sevntu.checkstyle/pull/990#issuecomment-1454337832 .

rnveach commented 1 year ago

run (sevntu-checks) CommitValidationTest.testCommitMessageHasProperStructure:189 Commit 3bace8653663caadfa481e797a5949d65871b823 message: "Merge 7f6451021ab39893feede91b0962095a3c408e01 into 1572f8c562482b7b75e9e8139686bfe386db3546\n" is invalid

Not sure if this is related to the commit depth or how we can fix this. @nrmancuso Do you have any ideas?

rnveach commented 1 year ago

@romani Please see https://github.com/sevntu-checkstyle/sevntu.checkstyle/pull/994#issuecomment-1454342303

romani commented 1 year ago

I pushed fix for no-exception on checkstyle https://github.com/checkstyle/contribution/commit/1eeef003464a29303d443bfc8031399dd8d3f3c0