Commit e764e19
authored
[Github] Remove separate tools checkout from pr-code workflows (llvm#159967)
These separate checkouts I believe were originally carried over from
when we were using pull_request_target. We needed two checkouts to
ensure we were not executing user supplied code. We kept them to ensure
we were using the latest version of the tools, but this was born mostly
out of a misunderstanding of how Github Actions works. All PRs directly
against main are executed as if merged into main, so already are using
the latest version of the tools no matter the branch point. Stacked PRs
still need to be rebased for changes to propagate but these files have
been pretty stable for the past two years or so and I can't imagine any
changes needed to keep things running on release/stacked PR branches.1 parent 290f5b5 commit e764e19
2 files changed
+5
-30
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
35 | | - | |
36 | | - | |
37 | | - | |
38 | | - | |
39 | | - | |
40 | | - | |
41 | | - | |
42 | | - | |
43 | | - | |
44 | | - | |
45 | | - | |
46 | | - | |
47 | | - | |
48 | 35 | | |
49 | 36 | | |
50 | 37 | | |
| |||
65 | 52 | | |
66 | 53 | | |
67 | 54 | | |
68 | | - | |
| 55 | + | |
69 | 56 | | |
70 | 57 | | |
71 | | - | |
| 58 | + | |
72 | 59 | | |
73 | 60 | | |
74 | 61 | | |
| |||
77 | 64 | | |
78 | 65 | | |
79 | 66 | | |
80 | | - | |
| 67 | + | |
81 | 68 | | |
82 | 69 | | |
83 | 70 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
46 | 46 | | |
47 | 47 | | |
48 | 48 | | |
49 | | - | |
50 | | - | |
51 | | - | |
52 | | - | |
53 | | - | |
54 | | - | |
55 | | - | |
56 | | - | |
57 | | - | |
58 | | - | |
59 | | - | |
60 | | - | |
61 | 49 | | |
62 | 50 | | |
63 | 51 | | |
| |||
70 | 58 | | |
71 | 59 | | |
72 | 60 | | |
73 | | - | |
| 61 | + | |
74 | 62 | | |
75 | 63 | | |
76 | 64 | | |
| |||
107 | 95 | | |
108 | 96 | | |
109 | 97 | | |
110 | | - | |
| 98 | + | |
111 | 99 | | |
112 | 100 | | |
113 | 101 | | |
| |||
0 commit comments