-
Notifications
You must be signed in to change notification settings - Fork 30
Github actions overhaul #652
Copy link
Copy link
Open
Labels
Developer ToolsConcerns tools used by maintainers to ensure the integrity of the packageConcerns tools used by maintainers to ensure the integrity of the packagePerformanceConcerns the performance/optimisation of the packageConcerns the performance/optimisation of the package
Metadata
Metadata
Assignees
Labels
Developer ToolsConcerns tools used by maintainers to ensure the integrity of the packageConcerns tools used by maintainers to ensure the integrity of the packagePerformanceConcerns the performance/optimisation of the packageConcerns the performance/optimisation of the package
Type
Fields
Give feedbackNo fields configured for issues without a type.
The existing actions suffer from two issues which I delineate below. For each issue, I will also comment on possible fixes or changes going forward.
1. Tests take too long
The regression tests against default and supporthook are done serially, meaning there are in total 170 tests to run per action at the time of writing. This means that the duration of the actions are akin to that of latex2e, despite being a much smaller project. There is also (perhaps) a configuration mishap within the yml file itself that invokes the runner twice for the same push for the purpose of a PR. To this end, we may consider some of these options:
2. What of benchmarks?
I'm unsure what the purpose of the benchmarking is, seeing as it compares the time taken for a variety of TeXLive versions against some commits in
main. Ever since the maintainership being transferred, the outputs of the benchmark test do not go anywhere, and are not accessible. In fact, #562 mentions combining the outputs for these, but it's unclear what the original author envisioned. To this end, we may consider:devbranch, with tweaks