19 Commits

Author SHA1 Message Date
Erwan MATHIEU
41526bfe70 Prepare for merge
CURA-11622
2024-12-04 15:01:15 +01:00
Erwan MATHIEU
6313a89cd6 Use uptodate install python script 2024-11-27 08:59:01 +01:00
Erwan MATHIEU
2b8a960421 Use most recent get-diff action 2024-11-22 16:06:03 +01:00
Erwan MATHIEU
3945465e50 Fix stupid deprecated GitHb actions
CURA-12093
2024-09-10 18:21:24 +02:00
Erwan MATHIEU
f4349be84a Fix diagnose script to match post script 2024-05-28 09:03:46 +02:00
Erwan MATHIEU
170c511568 Reduce permissions on printer-linter GitHub actions
Previously they had no permission set, so by default all write access. We now specify only the required permissions for each of them.
2024-05-23 11:49:35 +02:00
Saumya Jain
561a40d000 Implement GitComment class and update workflow files
A new GitComment class was implemented to replace Diagnostic for deleted file checks. As part of this change, both main workflow files (printer-linter-pr-diagnose.yml and printer-linter-pr-post.yml) have been updated to accommodate this new class. Also, reports now use 'comment.md' instead of 'fixes.yml'. All of this is ultimately geared at improving diagnostic functionality and allowing deleted file checks to output directly to a Git comment.

CURA-10903
2024-04-09 11:33:24 +02:00
Saumya Jain
b3a25893fc fix results to result
CURA-10903
2024-04-08 12:25:10 +02:00
Saumya Jain
c53e611c81 Update printer-linter workflow and fix file checks order
CURA-10903
2024-04-08 12:23:26 +02:00
Saumya Jain
13ad75a240 fixing command
CURA-10903
2024-04-08 10:52:17 +02:00
Saumya Jain
8b135b5562 Update command to retrieve deleted files
CURA-10903
2024-04-08 10:50:24 +02:00
Saumya Jain
2f8829c2d1 GIT_DIFF not necessarity have the info of deleted files
CURA-10903
2024-04-08 10:42:17 +02:00
Saumya Jain
38382eeec7 Add detection for deleted files in printer linter
This update adds a new check for deleted files in the printer linter. This will alert the user when a file has been deleted that could potentially disrupt upgrade scripts. An argument "--deleted" is also added to terminal.py to facilitate this new check. Additionally, the printer-linter version has been incremented to 0.1.2.

CURA-10903
2024-04-05 15:55:30 +02:00
Remco Burema
a1c01193da Move check so we can see if we even need python/pip.
Otherwise those if checks placed in the last commit won't even do anything if there _is_ a change in the printer definitions.
2023-02-16 14:50:31 +01:00
Remco Burema
04016373ae Prevent execution of spurious clean-up step, which then crashes.
The clean-up step for setting up Python and pip assumes that you do at least something with those programs -- and so that cache folders have been made, that need to be cleaned up. If nothing happens, the clean-up step then crashes on the non-existant cache folder.
2023-02-16 12:53:18 +01:00
jelle Spijker
8117087664 Revert "Fix multiline printer-linter PR suggestions"
This reverts commit 7820cc98
2022-12-13 11:30:13 +01:00
Jelle Spijker
9f3490893a
use master for clang-tidy-pr-comments 2022-12-10 16:05:07 +01:00
jspijker
7820cc9840 Fix multiline printer-linter PR suggestions 2022-12-02 10:28:42 +01:00
jspijker
32673bedea Add PR review workflows
Using the pull-request-comments-from-clang-tidy-reports GH action
https://github.com/marketplace/actions/pull-request-comments-from-clang-tidy-reports

Which is possible because the printer-linter auto
puts the suggested fixes in the same format as
clang-tidy.

It consists of two workflows, to ensure safe handling
of forks PR's. The first workflows is the analysis and
the suggested fixes, these are then upload to the
second workflow which will use the suggested fixes
to past comments on the PR
2022-11-21 13:23:04 +01:00