-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[FAST_BUILD] Introduce fast build #2014
Conversation
This approach allows us to test the most crucial changes in the CI in a fast way. As I'm not sure this is the best approach, and it adds some noise to the CI graph and the workflow file is bigger, I would like to hear some ideas, maybe there is a better approach. |
…ot available for PRs
If it's a PR, then everything depends on whether the PR title has a |
I don't understand what's going on 😢 |
Finally got it working. Today I learned GitHub env variables are strings, not some kind of GitHub magic, so, the "false" string evaluates to true (if then later used as a condition). What's cool (I suppose it's secure enough, but who knows), is that GitHub evaluates the expression after replacing env variables with values. |
@yuvipanda you might find this feature useful - it allows to significantly reduce build time in PRs. Please, review, if you have some time 🙂 |
I am merging this. I want to introduce new tests for run-hooks before we merge #2007. |
Describe your changes
Issue ticket if applicable
Checklist (especially for first-time contributors)