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

[Test PR] Revert "Looks like fuzzing is failing permanently again" #1

Closed
wants to merge 1 commit into from

Conversation

EliahKagan
Copy link
Owner

@EliahKagan EliahKagan commented Aug 23, 2024

This fork-internal pull request, which is for testing only and should not be merged here (a separate upstream PR would be opened if called for), reverts commit 3d90ab0 to investigate why fuzzing seems to be working when I trigger it via workflow_dispatch yet repeatedly failed when triggered with the pull_request trigger in GitoxideLabs#1536, per GitoxideLabs#1536 (comment).

@Byron
Copy link

Byron commented Aug 24, 2024

This PR seems to reproduce the issue I encountered. I think it's related to using the latest version of serde - maybe downgrading it via Cargo.lock would do the trick.

@EliahKagan
Copy link
Owner Author

EliahKagan commented Sep 15, 2024

It looks like this has to be tested in a PR on the upstream repository, because using that repository is hard-coded in the image. From this run:

#8 [2/5] RUN git clone --depth 1 https://github.com/Byron/gitoxide.git gitoxide
#8 sha256:3b146b60a92a660cccce81ae1557b6148b213196984c720682f15a74cfdd70da
#8 0.[185](https://github.com/EliahKagan/gitoxide/actions/runs/10867743127/job/30156857497?pr=1#step:4:186) Cloning into 'gitoxide'...
#8 DONE 2.2s
2024-09-15 03:25:12,382 - root - DEBUG - Stderr of command "git fetch origin refs/pull/1/merge" is: fatal: couldn't find remote ref refs/pull/1/merge
.
2024-09-15 03:25:12,382 - root - ERROR - Can not check out requested state refs/pull/1/merge. Using current repo state.

So I'm closing this here.

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