<feed xmlns='http://www.w3.org/2005/Atom'>
<title>llvm-project.git/.github/workflows/pr-code-format.yml, branch users/nico/python-2</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.
</subtitle>
<link rel='alternate' type='text/html' href='https://git.belthelziquor.com/llvm-project.git/'/>
<entry>
<title>[Github] Bump PR Code Format clang-format version (#141819)</title>
<updated>2025-05-28T19:16:01+00:00</updated>
<author>
<name>Aiden Grossman</name>
<email>aidengrossman@google.com</email>
</author>
<published>2025-05-28T19:16:01+00:00</published>
<link rel='alternate' type='text/html' href='https://git.belthelziquor.com/llvm-project.git/commit/?id=46f90165be92e08e059dcc07d42347cbf7446a0b'/>
<id>46f90165be92e08e059dcc07d42347cbf7446a0b</id>
<content type='text'>
As we have discussed in the past, we should be using the latest version.
This has not been updated in a while.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
As we have discussed in the past, we should be using the latest version.
This has not been updated in a while.</pre>
</div>
</content>
</entry>
<entry>
<title>[Github] Remove invalid comment from pr-code-format.yaml</title>
<updated>2025-04-27T04:12:54+00:00</updated>
<author>
<name>Aiden Grossman</name>
<email>aidengrossman@google.com</email>
</author>
<published>2025-04-27T04:12:54+00:00</published>
<link rel='alternate' type='text/html' href='https://git.belthelziquor.com/llvm-project.git/commit/?id=f11d46c1514d0ef4d3e92a96a9d330a9bc3164e1'/>
<id>f11d46c1514d0ef4d3e92a96a9d330a9bc3164e1</id>
<content type='text'>
The comment was originally added in
b3af755254599f474601d0d7b7374dcc36af279a, but
2898c3e0bbf904c68efb194525bcdfeedb3014c2 greatly simplified things
making the previous comment invalid. We no longer need to look at
diffing main versus the PR and whether we are doing a two dot or three
dot diff as we are just diffing the merge commit.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The comment was originally added in
b3af755254599f474601d0d7b7374dcc36af279a, but
2898c3e0bbf904c68efb194525bcdfeedb3014c2 greatly simplified things
making the previous comment invalid. We no longer need to look at
diffing main versus the PR and whether we are doing a two dot or three
dot diff as we are just diffing the merge commit.
</pre>
</div>
</content>
</entry>
<entry>
<title>[Github] Simplify Getting Changed Files in Code Formatting Workflow (#133023)</title>
<updated>2025-03-29T04:45:07+00:00</updated>
<author>
<name>Aiden Grossman</name>
<email>aidengrossman@google.com</email>
</author>
<published>2025-03-29T04:45:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.belthelziquor.com/llvm-project.git/commit/?id=2898c3e0bbf904c68efb194525bcdfeedb3014c2'/>
<id>2898c3e0bbf904c68efb194525bcdfeedb3014c2</id>
<content type='text'>
This patch changes getting changed files in the pr code format job to
just checking out the previous two commits (the merge commit and its
porent, the current commit latest in main), which allows us to just diff
the merge commit. This means we do not have to checkout the ref through
the merge base, which should save approximately a minute per job (or
much more in some cases where the PR is particularly out of date).</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch changes getting changed files in the pr code format job to
just checking out the previous two commits (the merge commit and its
porent, the current commit latest in main), which allows us to just diff
the merge commit. This means we do not have to checkout the ref through
the merge base, which should save approximately a minute per job (or
much more in some cases where the PR is particularly out of date).</pre>
</div>
</content>
</entry>
<entry>
<title>[Github] Change to step-security fork of changed actions</title>
<updated>2025-03-15T20:56:29+00:00</updated>
<author>
<name>Aiden Grossman</name>
<email>aidengrossman@google.com</email>
</author>
<published>2025-03-15T20:56:29+00:00</published>
<link rel='alternate' type='text/html' href='https://git.belthelziquor.com/llvm-project.git/commit/?id=6616acd80cd91a0075e3cd481bb9a6d82fd4ea9e'/>
<id>6616acd80cd91a0075e3cd481bb9a6d82fd4ea9e</id>
<content type='text'>
The tj-actions/changed-files repo has been taken down to the security
incident
(https://www.stepsecurity.io/blog/harden-runner-detection-tj-actions-changed-files-action-is-compromised).

This patch moves these jobs over step-security's fork, which has been
loosely audited and has had the malicious commits removed. This is
mainly intended as a stop-gap to get these actions running again while
we figure out the best path forward.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The tj-actions/changed-files repo has been taken down to the security
incident
(https://www.stepsecurity.io/blog/harden-runner-detection-tj-actions-changed-files-action-is-compromised).

This patch moves these jobs over step-security's fork, which has been
loosely audited and has had the malicious commits removed. This is
mainly intended as a stop-gap to get these actions running again while
we figure out the best path forward.
</pre>
</div>
</content>
</entry>
<entry>
<title>[Github] Prefer ubuntu-24.04 over ubuntu-latest (#129936)</title>
<updated>2025-03-06T18:32:02+00:00</updated>
<author>
<name>Aiden Grossman</name>
<email>aidengrossman@google.com</email>
</author>
<published>2025-03-06T18:32:02+00:00</published>
<link rel='alternate' type='text/html' href='https://git.belthelziquor.com/llvm-project.git/commit/?id=0ce4b685d2f46b389887aeb8d1d16d5d15f7ecce'/>
<id>0ce4b685d2f46b389887aeb8d1d16d5d15f7ecce</id>
<content type='text'>
This patch replaces all instances of ubuntu-latest with ubuntu-24.04
(outside of the entries in libc++) based on the guidelines in the LLVM
CI best practices doc (https://llvm.org/docs/CIBestPractices.html).</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch replaces all instances of ubuntu-latest with ubuntu-24.04
(outside of the entries in libc++) based on the guidelines in the LLVM
CI best practices doc (https://llvm.org/docs/CIBestPractices.html).</pre>
</div>
</content>
</entry>
<entry>
<title>[Github] Hash Pin Actions in Most Workflows (#129486)</title>
<updated>2025-03-03T16:37:20+00:00</updated>
<author>
<name>Aiden Grossman</name>
<email>aidengrossman@google.com</email>
</author>
<published>2025-03-03T16:37:20+00:00</published>
<link rel='alternate' type='text/html' href='https://git.belthelziquor.com/llvm-project.git/commit/?id=705decc860f1e7aa73476463ad98ce2ea293f7da'/>
<id>705decc860f1e7aa73476463ad98ce2ea293f7da</id>
<content type='text'>
This patch haspins all actions in most of the LLVM Github workflows.
This is something we try to do, but no one has gone through and combed
through all of the workflows before this patch.

Notably, this patch does not bump any major versions of actions just in
case there are subtle breaking changes introduced between versions that
could impact us. Also, this patch omits the libc/libc++ workflows so
that they can be split into separate PRs for the respective subproject
maintainers to review.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch haspins all actions in most of the LLVM Github workflows.
This is something we try to do, but no one has gone through and combed
through all of the workflows before this patch.

Notably, this patch does not bump any major versions of actions just in
case there are subtle breaking changes introduced between versions that
could impact us. Also, this patch omits the libc/libc++ workflows so
that they can be split into separate PRs for the respective subproject
maintainers to review.</pre>
</div>
</content>
</entry>
<entry>
<title>[ci] Bump the version of clang-format used in the CI (#119915)</title>
<updated>2025-01-07T21:40:22+00:00</updated>
<author>
<name>Louis Dionne</name>
<email>ldionne.2@gmail.com</email>
</author>
<published>2025-01-07T21:40:22+00:00</published>
<link rel='alternate' type='text/html' href='https://git.belthelziquor.com/llvm-project.git/commit/?id=858f025a00fd107e9b8f97b630028d40c0c68725'/>
<id>858f025a00fd107e9b8f97b630028d40c0c68725</id>
<content type='text'>
The version of clang-format we use in the CI to format all PRs is a bit
outdated, leading to some confusion when the CI job produces different
output from what people have locally.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The version of clang-format we use in the CI to format all PRs is a bit
outdated, leading to some confusion when the CI job produces different
output from what people have locally.</pre>
</div>
</content>
</entry>
<entry>
<title>[Github] Cancel previous in-progress code formatting jobs (#106701)</title>
<updated>2024-08-30T21:43:15+00:00</updated>
<author>
<name>Aiden Grossman</name>
<email>aidengrossman@google.com</email>
</author>
<published>2024-08-30T21:43:15+00:00</published>
<link rel='alternate' type='text/html' href='https://git.belthelziquor.com/llvm-project.git/commit/?id=1293ab35e406e8b50030335ccf98580a7b719ff5'/>
<id>1293ab35e406e8b50030335ccf98580a7b719ff5</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>[Github] Set code formatting job timeout to 30 minutes (#106674)</title>
<updated>2024-08-30T08:08:38+00:00</updated>
<author>
<name>Aiden Grossman</name>
<email>aidengrossman@google.com</email>
</author>
<published>2024-08-30T08:08:38+00:00</published>
<link rel='alternate' type='text/html' href='https://git.belthelziquor.com/llvm-project.git/commit/?id=54916e5784d0fd928ef5b28b2eb99726a395cb3f'/>
<id>54916e5784d0fd928ef5b28b2eb99726a395cb3f</id>
<content type='text'>
This patch sets the timeout of the code formatting job to 30 minutes.
The job is currently failing in specific circumstances and needs to be
reworked, but as a temp hack, change the timeout to 30 minutes so that
we can catch these jobs before they hit the Github Actions timeout limit
of six hours.

Somewhat (hackily) alleviates #79661 slightly.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch sets the timeout of the code formatting job to 30 minutes.
The job is currently failing in specific circumstances and needs to be
reworked, but as a temp hack, change the timeout to 30 minutes so that
we can catch these jobs before they hit the Github Actions timeout limit
of six hours.

Somewhat (hackily) alleviates #79661 slightly.</pre>
</div>
</content>
</entry>
<entry>
<title>[GitHub] Run format job on stacked PRs (#102957)</title>
<updated>2024-08-14T12:10:57+00:00</updated>
<author>
<name>Tobias Stadler</name>
<email>mail@stadler-tobias.de</email>
</author>
<published>2024-08-14T12:10:57+00:00</published>
<link rel='alternate' type='text/html' href='https://git.belthelziquor.com/llvm-project.git/commit/?id=98119718603c6e623b34e3d2160b65b26d9bfcdc'/>
<id>98119718603c6e623b34e3d2160b65b26d9bfcdc</id>
<content type='text'>
Currently the formatter only runs for the main branch, which prevents
the formatter from running for stacked PRs, which have to target user
branches instead of main.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Currently the formatter only runs for the main branch, which prevents
the formatter from running for stacked PRs, which have to target user
branches instead of main.</pre>
</div>
</content>
</entry>
</feed>
