-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update workflow #61
Comments
I also want to an initial step to halt simultaneous actions, somehow prioritize the first one, race condition is fine |
Also stop creating so many commit when not needed, only commit if something changed
|
I think I should take an even bigger step back -- The entire workflow is convoluted. I'm trying to do everything at once, so let's break it down. AbstractWhen changed:
Change can be
1. Ensure it still works
2. Update information
3. PublishIf the version is bumped and tests pass then we publish.
Current setupCurrently, these 2 of these steps happen for all 5 of my general packages (only Python) with every commit. Special treatmentI think it's a mistake to treat my dependents differently. Workflow should run when any dependency is updated. Re-sync Library when Packager is publishedSince Packager isn't a dependency for Library, Library won't be triggered when Packager is published. Version lockingOption 1 -- Commit on success:
Option 2 -- Commit on fail:
Behavior in my environment on publish:C depends on B Option 1 -- Commit on success:
Recursive workflows, not ideal Option 2 -- Commit on fail:
If we can make indirect dependencies trigger then this seems better! Lock indirect dependencies?Probably don't bother, would mess up the dependency chain cause it would look like my package directly requires all indirect dependencies, it would "flatten" the chain. Private dependenciesI'll need to support private dependencies in the future, perhaps wait with this for now Plan
|
I want to support [LINUX] and [WINDOWS] for when I only want to test on a single quick instance.
Hmm maybe being able to write [3.8] could be good too.
What about any combination?
The text was updated successfully, but these errors were encountered: