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

[build-tools] Do not add repack.yml to project directory #496

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

Conversation

sjchmiela
Copy link
Contributor

@sjchmiela sjchmiela commented Jan 16, 2025

Why

If developers have requireCommit: true in their eas.json configuration, when they try to publish an update with Workflows it fails, because there are uncommitted files in repository — __eas/repack.yml.

How

Instead of copying repack.yml into people's projects we're going to define steps in code.

Test Plan

Tested manually.

Copy link

linear bot commented Jan 16, 2025

Base automatically changed from stanley/eng-14684-add-steps-to-build-job to main January 17, 2025 15:00
@sjchmiela sjchmiela force-pushed the stanley/eng-14735-stop-adding-repackyml-to-peoples-project-directories branch from 4bf33b5 to 8e42a87 Compare January 18, 2025 15:20
@sjchmiela sjchmiela force-pushed the stanley/eng-14735-stop-adding-repackyml-to-peoples-project-directories branch from 8e42a87 to 009f854 Compare January 23, 2025 14:10
@sjchmiela sjchmiela force-pushed the stanley/eng-14735-stop-adding-repackyml-to-peoples-project-directories branch from 801ab16 to bc84adc Compare January 23, 2025 16:31
@sjchmiela sjchmiela marked this pull request as ready for review January 23, 2025 16:35
@sjchmiela sjchmiela requested a review from szdziedzic as a code owner January 23, 2025 16:35
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