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

Fix race conditions #330

Merged
merged 4 commits into from
Feb 6, 2024
Merged

Fix race conditions #330

merged 4 commits into from
Feb 6, 2024

Conversation

gitfool
Copy link
Contributor

@gitfool gitfool commented Feb 6, 2024

Fixes #321.

Target conditions and git info cache sufficient for incremental build.
Continue on error as eventually successful.
Continue on error as eventually successful.
Continue on error as same as no cache.
@CLAassistant
Copy link

CLAassistant commented Feb 6, 2024

CLA assistant check
All committers have signed the CLA.

@gitfool
Copy link
Contributor Author

gitfool commented Feb 6, 2024

@kzu PTAL; I've done a fair bit of local testing and would love to get this approved, merged and released. 🙏

@kzu
Copy link
Member

kzu commented Feb 6, 2024

I like how you changed properties with file reading to items via read lines 👌.

The non -central location for the cache has to do with submodules and projects in the same solution coming from them.

@kzu kzu enabled auto-merge (rebase) February 6, 2024 23:51
@kzu kzu merged commit 5eb6b33 into devlooped:main Feb 6, 2024
5 checks passed
@gitfool gitfool deleted the gh321 branch February 6, 2024 23:51
@devlooped devlooped locked and limited conversation to collaborators Sep 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Weird race condition with targets
3 participants