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

chore(aft): Custom GitHub actions config #3628

Merged
merged 1 commit into from
Aug 31, 2023
Merged

Conversation

dnys1
Copy link
Contributor

@dnys1 dnys1 commented Aug 28, 2023

Currently, aft generate workflows checks for the presence of a workflow.yaml file next to a pubspec.yaml to know if we should generate a GitHub action workflow or copy over the custom one. This leads to drift, though, since dependabot does not search these directories by default.

We could update Dependabot to search these directories, but instead we add a setting to the aft config to say that the workflow which exists is custom and should not be overridden so that the information is not copied in two places.

@dnys1 dnys1 requested a review from a team as a code owner August 28, 2023 18:43
Jordan-Nelson
Jordan-Nelson previously approved these changes Aug 29, 2023
Equartey
Equartey previously approved these changes Aug 29, 2023
Currently, `aft generate workflows` checks for the presence of a `workflow.yaml` file next to a `pubspec.yaml` to know if we should generate a GitHub action workflow or copy over the custom one. This leads to drift, though, since dependabot does not search these directories by default.

We could update Dependabot to search these directories, but instead we add a setting to the aft config to say that the workflow which exists is custom and should not be overridden so that the information is not copied in two places.
@dnys1 dnys1 dismissed stale reviews from Equartey and Jordan-Nelson via 8672967 August 31, 2023 15:09
@dnys1 dnys1 force-pushed the chore/aft/custom-gh-actions branch from dd14501 to 8672967 Compare August 31, 2023 15:09
@dnys1 dnys1 merged commit fca9d4d into main Aug 31, 2023
4 checks passed
@dnys1 dnys1 deleted the chore/aft/custom-gh-actions branch August 31, 2023 18:07
dnys1 added a commit that referenced this pull request Sep 11, 2023
Currently, `aft generate workflows` checks for the presence of a `workflow.yaml` file next to a `pubspec.yaml` to know if we should generate a GitHub action workflow or copy over the custom one. This leads to drift, though, since dependabot does not search these directories by default.

We could update Dependabot to search these directories, but instead we add a setting to the aft config to say that the workflow which exists is custom and should not be overridden so that the information is not copied in two places.
dnys1 added a commit that referenced this pull request Sep 11, 2023
Currently, `aft generate workflows` checks for the presence of a `workflow.yaml` file next to a `pubspec.yaml` to know if we should generate a GitHub action workflow or copy over the custom one. This leads to drift, though, since dependabot does not search these directories by default.

We could update Dependabot to search these directories, but instead we add a setting to the aft config to say that the workflow which exists is custom and should not be overridden so that the information is not copied in two places.
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.

3 participants