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

New CI for DiffTool.java #192

Merged
merged 1 commit into from
Oct 5, 2024
Merged

Conversation

relentless-pursuit
Copy link
Collaborator

Resolves #186

Copy link
Member

@romani romani left a comment

Choose a reason for hiding this comment

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

items:

.github/workflows/DiffTool-CI.yml Outdated Show resolved Hide resolved
.github/workflows/DiffTool-CI.yml Outdated Show resolved Hide resolved
Copy link
Member

@romani romani left a comment

Choose a reason for hiding this comment

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

item:

.ci-config/config.xml Outdated Show resolved Hide resolved
Copy link
Member

@romani romani left a comment

Choose a reason for hiding this comment

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

items:

.github/workflows/DiffTool-CI.yml Outdated Show resolved Hide resolved
.github/workflows/DiffTool-CI.yml Outdated Show resolved Hide resolved
Copy link
Member

@romani romani left a comment

Choose a reason for hiding this comment

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

items:

.github/workflows/DiffTool-CI.yml Outdated Show resolved Hide resolved
Copy link
Member

@romani romani left a comment

Choose a reason for hiding this comment

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

item:

- name: diff-java-tool
scm: git
url: file:///home/runner/work/test-configs/test-configs/diff-java-tool
reference: main
Copy link
Member

Choose a reason for hiding this comment

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

continuation of #192 (comment)

try:

    url: file:///home/runner/work/test-configs
    reference: main

as it most likely neded folder with ".github"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done, waiting for CI

Copy link
Member

@romani romani left a comment

Choose a reason for hiding this comment

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

item:

.github/workflows/DiffTool-CI.yml Outdated Show resolved Hide resolved
Copy link
Member

@romani romani left a comment

Choose a reason for hiding this comment

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

items:

.github/workflows/ci-difftool.yml Outdated Show resolved Hide resolved
Copy link
Member

@romani romani left a comment

Choose a reason for hiding this comment

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

items:

.github/workflows/ci-diff-java-tool.yml Outdated Show resolved Hide resolved
@romani
Copy link
Member

romani commented Oct 5, 2024

https://github.com/checkstyle/test-configs/actions/runs/11196866258/job/31126176975?pr=192#step:12:342

[main] INFO com.github.checkstyle.difftool.DiffTool - Testing Checkstyle started
[main] INFO com.github.checkstyle.difftool.DiffTool - Cloning git repository 'diff-java-tool' to repositories/diff-java-tool folder ...
Cloning into 'repositories/diff-java-tool'...
hint: Using 'master' as the name for the initial branch. This default branch name
hint: is subject to change. To configure the initial branch name to use in all
hint: of your new repositories, which will suppress this warning, call:
hint:
hint: 	git config --global init.defaultBranch <name>
hint:
hint: Names commonly chosen instead of 'master' are 'main', 'trunk' and
hint: 'development'. The just-created branch can be renamed via this command:
hint:
hint: 	git branch -m <name>
Note: switching to 'dd6eae8136fd7c8c29127471bf5b94d95a2a0f53'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

[main] INFO com.github.checkstyle.difftool.DiffTool - Cloning git repository 'diff-java-tool' - completed

[main] INFO com.github.checkstyle.difftool.DiffTool - Running command: git fetch origin main:main
fatal: couldn't find remote ref main
Error: Exception in thread "main" com.github.checkstyle.difftool.DiffTool$CommandExecutionException: Command execution failed (Exit code: 128)
	at com.github.checkstyle.difftool.DiffTool.executeCmd(DiffTool.java:1796)
	at com.github.checkstyle.difftool.DiffTool.fetchAdditionalData(DiffTool.java:966)
	at com.github.checkstyle.difftool.DiffTool.cloneRepository(DiffTool.java:879)
	at com.github.checkstyle.difftool.DiffTool.generateCheckstyleReport(DiffTool.java:626)
	at com.github.checkstyle.difftool.DiffTool.launchCheckstyleReport(DiffTool.java:562)
	at com.github.checkstyle.difftool.DiffTool.main(DiffTool.java:132)
Error: Process completed with exit code 1.

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.

2 participants