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

MultiCI: Abstract state reading #18

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Nava2
Copy link
Owner

@Nava2 Nava2 commented Jan 9, 2025

There is no functional changes present, access is moved behind small wrapper class to provide a simple abstraction to be replaced.

Currently, we implement this in terms of Github Actions -- for obvious reasons. In the future, this enables implementation in other, non-Github CI environments.

#22 contains all usages replaced directly.

Part of gradle#502.

Copy link
Owner Author

Nava2 commented Jan 9, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

This adds a new pattern for abstracting away the "github action"-specific code within the github actions. The
overall intention is to perform this change for all of the `@actions/xxx` libraries in use to enable
implementation of these features within other environments (e.g. Jenkins, BuildKite, GitLab).

Part of gradle#502.
@Nava2 Nava2 force-pushed the kevin/abstract-gha/abstract-state-reading branch from b2d72ed to 842c685 Compare January 9, 2025 18:19
Nava2 added a commit that referenced this pull request Jan 9, 2025
Following #18, we finish replacing all usages. This was split to enable easier
review and acknowledge there's a couple small behaviour changes using `true.toString()` for example.

Part of gradle#502.
Nava2 added a commit that referenced this pull request Jan 9, 2025
Following #18, we finish replacing all usages. This was split to enable easier
review and acknowledge there's a couple small behaviour changes using `true.toString()` for example.

Part of gradle#502.
Nava2 added a commit that referenced this pull request Jan 10, 2025
Following #18, we finish replacing all usages. This was split to enable easier
review and acknowledge there's a couple small behaviour changes using `true.toString()` for example.

Part of gradle#502.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant