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

MegaLinter: Checkout the right merge commit #1617

Merged
merged 1 commit into from Dec 8, 2022

Conversation

vkucera
Copy link
Collaborator

@vkucera vkucera commented Dec 8, 2022

Running on the merge commit is necessary to get the right list of modified files.
@TimoWilken

Problem with listing the modified files reported: oxsecurity/megalinter#2125

@vkucera vkucera marked this pull request as draft December 8, 2022 00:40
@vkucera vkucera closed this Dec 8, 2022
@vkucera vkucera reopened this Dec 8, 2022
@vkucera vkucera changed the title MegaLinter: Run on pull_request instead of pull_request_target MegaLinter: Checkout the right merge commit Dec 8, 2022
Running on the merge commit is necessary to get the right list of modified files.
@vkucera vkucera marked this pull request as ready for review December 8, 2022 13:49
@TimoWilken TimoWilken merged commit b370eb6 into AliceO2Group:master Dec 8, 2022
@vkucera vkucera deleted the mega branch December 8, 2022 14:12
MrWhatZitToYaa pushed a commit to MrWhatZitToYaa/O2Physics that referenced this pull request Dec 20, 2022
Running on the merge commit is necessary to get the right list of modified files.
ercolessi pushed a commit to ercolessi/O2Physics that referenced this pull request Jan 9, 2023
Running on the merge commit is necessary to get the right list of modified files.
GijsvWeelden pushed a commit to GijsvWeelden/O2Physics that referenced this pull request Jan 13, 2023
Running on the merge commit is necessary to get the right list of modified files.
cimordas pushed a commit to cimordas/O2Physics that referenced this pull request Jan 20, 2023
Running on the merge commit is necessary to get the right list of modified files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants