Skip to content

ci: setup autofix#1679

Merged
spawnia merged 4 commits into
webonyx:masterfrom
simPod:pat
Apr 4, 2025
Merged

ci: setup autofix#1679
spawnia merged 4 commits into
webonyx:masterfrom
simPod:pat

Conversation

@simPod
Copy link
Copy Markdown
Collaborator

@simPod simPod commented Mar 12, 2025

No description provided.

@simPod simPod force-pushed the pat branch 2 times, most recently from 4e8545b to f5d9fb8 Compare March 12, 2025 10:02
@simPod
Copy link
Copy Markdown
Collaborator Author

simPod commented Mar 12, 2025

for some reason the workflow is not triggered even though PAT is set 🤔

Comment thread .github/workflows/autoformat.yml Outdated
Copy link
Copy Markdown
Collaborator

@spawnia spawnia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great find! I guess I will have to update like a dozen repos with this when it proves to work well 😅

Comment thread .github/workflows/autoformat.yml Outdated
Comment thread .github/workflows/autoformat.yml Outdated
@spawnia
Copy link
Copy Markdown
Collaborator

spawnia commented Apr 2, 2025

Can we make it so Static Analysis and Test checks run on the fix commit?

image

@simPod
Copy link
Copy Markdown
Collaborator Author

simPod commented Apr 2, 2025

@spawnia it's running

image

@simPod
Copy link
Copy Markdown
Collaborator Author

simPod commented Apr 2, 2025

It actually has permissions for other users as well 🤔 #1694

@simPod
Copy link
Copy Markdown
Collaborator Author

simPod commented Apr 2, 2025

The only problem I see is that actual checks are not visible in jobs

image image

the solution is to somewhat dupe em I guess

@simPod simPod force-pushed the pat branch 3 times, most recently from d048661 to 6a00c1e Compare April 2, 2025 08:53
@autofix-troubleshooter
Copy link
Copy Markdown

Hi! I'm the autofix logoautofix.ci troubleshooter bot.

It looks like you correctly set up a CI job that uses the autofix.ci GitHub Action, but the autofix.ci GitHub App has not been installed for this repository. This means that autofix.ci unfortunately does not have the permissions to fix this pull request. If you are the repository owner, please install the app and then restart the CI workflow! 😃

Copy link
Copy Markdown
Collaborator Author

@simPod simPod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It worked because I was still somewhat involved (commits)

image

we have to wait for installation.

@simPod simPod force-pushed the pat branch 2 times, most recently from 65e70bc to 06a3007 Compare April 2, 2025 09:03
@simPod simPod changed the title testing PAT ci: setup autofix Apr 2, 2025
@simPod simPod marked this pull request as ready for review April 2, 2025 09:03
Comment thread .github/workflows/autoformat.yml Outdated
@simPod simPod force-pushed the pat branch 6 times, most recently from e765231 to 4629f3a Compare April 4, 2025 14:08
@simPod
Copy link
Copy Markdown
Collaborator Author

simPod commented Apr 4, 2025

@spawnia the app expects a single artifact autofix.ci.zip, therefore I had to merge it all into a single job to support multiple style violations but I'm fine with it.

@spawnia spawnia merged commit 660a55f into webonyx:master Apr 4, 2025
18 checks passed
@spawnia
Copy link
Copy Markdown
Collaborator

spawnia commented Apr 4, 2025

Thank you @simPod for digging into this! I hope it is going to work well in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants