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

Add Windows zip #375

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

Conversation

ShiranAvidov
Copy link

@ShiranAvidov ShiranAvidov commented Jul 24, 2023

I created a new Build and Archive for Windows, so each release will have Windows tar.gz and zip files.

#373

@ShiranAvidov ShiranAvidov requested a review from a team as a code owner July 24, 2023 14:30
Copy link
Member

@jpkrohling jpkrohling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would be OK in giving this a try for v0.83.0. I would like to avoid merging this for the next release though (0.82.0), as it has the potential to break the already late current release.

@ShiranAvidov
Copy link
Author

@jpkrohling Is it possible to merge it to the next version (0.84.0)?

@jpkrohling
Copy link
Member

I'll close and open this PR again, in an attempt to retrigger the CI: it has failed with a disk space issue, which I'm hoping is not tied to this PR.

@jpkrohling jpkrohling closed this Aug 28, 2023
@jpkrohling jpkrohling reopened this Aug 28, 2023
@ShiranAvidov
Copy link
Author

@jpkrohling no space left on device

@jpkrohling
Copy link
Member

Can you check what are the disk requirements before and after this change? Are we just too close to the limit on the main branch, or is this change here causing too much usage?

.goreleaser.yaml Outdated Show resolved Hide resolved
@ShiranAvidov
Copy link
Author

@codeboten @jpkrohling I added partial to the yaml.

@ShiranAvidov
Copy link
Author

@jpkrohling @codeboten all checks have passed...

@atoulme
Copy link
Contributor

atoulme commented Oct 19, 2023

would you please look at the conflicts?

@bogdandrutu
Copy link
Member

Needs a rebase

@atoulme
Copy link
Contributor

atoulme commented Apr 26, 2024

@ShiranAvidov please let us know if you would like to work to resolve conflicts on this PR, otherwise we will close it for inactivity in 30 days.

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.

None yet

5 participants