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.

Sign up for GitHub

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

Jump to bottom

test: running lint on pull request #101

Merged

notdanilo merged 1 commit into main from test/lint-on-pull-request

Nov 29, 2024

Contributor

notdanilo commented Nov 29, 2024

Status	Type	⚠️ Core Change	Issue
Ready	Tooling	No	None

Problem

Wrong assumption that caches was causing issues while the code was being tested on pull_request_target
Linting is running on pull_request_target and not on pull_request

Solution

Bring caching back
Make it run on pull_request to lint incoming changes.


          test: running lint on pull request

43370c4

notdanilo merged commit 5283112 into main

2 checks passed

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

Labels

None yet