-
Notifications
You must be signed in to change notification settings - Fork 140
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
Error: The head commit for this pull_request event is not ahead of the base commit #42
Comments
Yes this probably should be an option if the author is around. In the mean time you can ignore the error of the step I believe with github actions |
I used another way to get modified files that is not based on tree relation between commits: API_URL=$(jq --raw-output .pull_request._links.self.href "$GITHUB_EVENT_PATH")
FILES=$(curl --request GET \
--url "$API_URL/files?per_page=100" \
--header "accept: application/vnd.github.v3+json" \
--header "authorization: token ${{secrets.GITHUB_TOKEN}}")
echo $FILES
CHANGED_FILES=$(echo $FILES | jq '.[] | select(.status == "added" or .status == "modified") | .filename') |
This is working for you? Running this locally I am getting:
|
Use the most popular action (249 stars at the moment) to retrieve the list of changed files, because the action we used before only works fine for branches forked off the tip of the master branch (fast-forward): jitterbit/get-changed-files#42
IceTDrinker, can you suggest a resource on how to ignore the error? |
@merobi-hub I'm guessing they're suggesting something like this? that's what I'm about to try
|
Error output in GitHub action pushed me to create this issue:
./.github/workflows/check-codestyle.yml
The text was updated successfully, but these errors were encountered: