Skip to content

MegaLinter: Upload artifacts with formatted files#5526

Merged
ddobrigk merged 1 commit intoAliceO2Group:masterfrom
vkucera:megalinter
Apr 15, 2024
Merged

MegaLinter: Upload artifacts with formatted files#5526
ddobrigk merged 1 commit intoAliceO2Group:masterfrom
vkucera:megalinter

Conversation

@vkucera
Copy link
Collaborator

@vkucera vkucera commented Apr 8, 2024

Enable uploading artifacts with formatted files https://megalinter.io/latest/reporters/UpdatedSourcesReporter/

MegaLinter first runs the formatting and then the rest of linters. If those linters find errors, the line numbers they report in the log refer to the formatted files, not to the committed files.
The artifacts allow to download the formatted files and find the exact location reported by the linters.

@vkucera vkucera requested review from a team, alibuild, ddobrigk, iarsene, jgrosseo and ktf as code owners April 8, 2024 17:58
@vkucera
Copy link
Collaborator Author

vkucera commented Apr 10, 2024

@ddobrigk , does it look good to you?

@ddobrigk ddobrigk merged commit af9716e into AliceO2Group:master Apr 15, 2024
@vkucera vkucera deleted the megalinter branch April 15, 2024 09:27
christianreckziegel pushed a commit to christianreckziegel/O2Physics that referenced this pull request May 2, 2024
hahassan7 pushed a commit to hahassan7/O2Physics that referenced this pull request May 17, 2024
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.

2 participants