Uh oh!
There was an error while loading. Please reload this page.
PWGDQ: fixing the configuration file for the fit - #983
Conversation
lucamicheletti93
commented
Aug 1, 2022
Dear @TimoWilken, I'm having this issue with MegaLinter, in particular the issue seems to be related to "PR not clear" but I don't know how this can be fixed. Do you have any suggestion? |
TimoWilken
commented
Aug 1, 2022
Hi @lucamicheletti93, the auto-fixes applied by MegaLinter are in a pull request in your fork of O2Physics: lucamicheletti93#2. |
lucamicheletti93
commented
Aug 1, 2022
Dear @TimoWilken , thanks a lot! This means that I don't need to do any action? |
TimoWilken
commented
Aug 1, 2022
No, you should look at the pull request I linked to, and merge it. Only then will the MegaLinter check in this pull request succeed. Alternatively, you can make similar formatting fixes manually. |
lucamicheletti93
commented
Aug 8, 2022
Dear @TimoWilken , thanks for the explanation! I have approved the changes but it seems that the PR is not yet merged. How should I proceed? Would it be better to apply by hand the changes and do another commit? |
Error while checking build/O2Physics/o2 for 1d56205 at 2022-08-11 03:30: Full log here. |
TimoWilken
commented
Aug 15, 2022
Hi @lucamicheletti93, the PR lucamicheletti93#2 is in your own repository, so you need to merge it directly using the large "Squash and merge" button near the bottom of the page I linked above. Only approving it will not do anything, because the pull request is in your own repository. Alternatively, if you prefer, you can look at the changes here and apply them manually. You can also download the patch file from here and pass it to the |
lucamicheletti93
commented
Aug 17, 2022
Dear @TimoWilken , thanks again for the explanation! I did "Squash and merge" and the test started |
Fixing the configuration file for the fit