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: simplify scripts and ensure we are able to rerun #927

Merged
merged 2 commits into from Oct 29, 2022

Conversation

rnveach
Copy link
Contributor

@rnveach rnveach commented Oct 29, 2022

identified from https://github.com/sevntu-checkstyle/sevntu.checkstyle/blob/master/.ci/validation.sh#L20 when trying to run locally and running into issues,

sevntu.checkstyle/ $ ./.ci/validation.sh eclipse-cs

.....
fatal: destination path 'eclipse-cs' already exists and is not an empty directory.


sevntu.checkstyle/.ci-temp/eclipse-cs/ $ git status

HEAD detached at 10.0.0
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

    modified:  docs/partials/releases/5.0.0final/release_notes.html
    modified:  docs/sitemap.xml

no changes added to commit (use "git add" and/or "git commit -a")

@@ -203,7 +189,7 @@ pmd-elasticsearch-lombok-ast)
--non-recursive org.codehaus.mojo:exec-maven-plugin:1.3.1:exec)
echo CS_version: ${CS_POM_VERSION}
echo SEVNTU_version: ${SEVNTU_POM_VERSION}
checkout_from "-b issue-529 https://github.com/nmancus1/contribution "
checkout_from "https://github.com/checkstyle/contribution.git "
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this may have been forgotten to undo.
See #828 (comment)

@rnveach
Copy link
Contributor Author

rnveach commented Oct 29, 2022

Confirmed working on my local.

sevntu.checkstyle/ $ ./.ci/validation.sh eclipse-cs

....
[INFO] ------------------------------------------------------------------------
Target project eclipse-cs is already cloned, latest changes will be fetched and reset
HEAD is now at 6ab343b9 config: release 10.0.0
Eclipse-cs tag: 10.0.0
HEAD is now at 6ab343b9 config: release 10.0.0
[INFO] Error stacktraces are turned on.
....
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  17.543 s
[INFO] Finished at: 2022-10-29T13:00:55-04:00
[INFO] ------------------------------------------------------------------------

@rnveach
Copy link
Contributor Author

rnveach commented Oct 29, 2022

For travis, s390x is having issues starting. That is why I was trying to run this job locally.

@romani romani merged commit 03d5a0b into sevntu-checkstyle:master Oct 29, 2022
@rnveach rnveach deleted the repeatability branch October 29, 2022 19:36
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.

None yet

2 participants