Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Autofix GitHub Action #1202

Merged
merged 3 commits into from
Oct 13, 2023
Merged

Autofix GitHub Action #1202

merged 3 commits into from
Oct 13, 2023

Conversation

filiptronicek
Copy link
Member

Description

This PR runs Autofix inside a GitHub Action, which results in us having more control over how it runs. Specifically, the Pull Requests raised will actually result from branches inside the repository, meaning that CI will be happy and automated tests will run correctly.

Related Issue(s)

Fixes EXP-486

@filiptronicek filiptronicek self-assigned this Oct 12, 2023
@filiptronicek filiptronicek marked this pull request as ready for review October 12, 2023 20:59
@filiptronicek filiptronicek requested review from a team as code owners October 12, 2023 20:59

- name: Run autofix
run: |
npm -g autofix
Copy link
Member

Choose a reason for hiding this comment

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

nit: would be cool to tie versions here somehow, so we have at least some amount of control over what code we pass our GITHUB_TOKEN to... 😕 Can we maybe hard-code a specific version of autofix at least?

But I guess we are already doing this somewhere else, and this is merely a better way to execute it, so partly out of scope here. 😕

Copy link
Member Author

Choose a reason for hiding this comment

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

@geropl thanks for pointing this out! I will keep Autofix updated and have now pinned the version to the latest one 🙏

Copy link
Member

@geropl geropl left a comment

Choose a reason for hiding this comment

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

✔️ to unblock

See the comment above for a general security concern/nit

@filiptronicek filiptronicek merged commit c56dd0a into main Oct 13, 2023
4 checks passed
@filiptronicek filiptronicek deleted the ft/autofix-gha branch October 13, 2023 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants