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

actions: Create release on push #59

Merged
merged 1 commit into from
Dec 30, 2021
Merged

actions: Create release on push #59

merged 1 commit into from
Dec 30, 2021

Conversation

dracc
Copy link
Contributor

@dracc dracc commented Dec 13, 2021

No description provided.

@MasonT8198
Copy link
Contributor

Looks good to me @mborgerson

pushd dist/extract-xiso_Win32_Debug
zip -r ../extract-xiso-win32-debug.zip *
popd
pushd dist/extract-xiso_Win64_Release
Copy link
Contributor

Choose a reason for hiding this comment

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

Should the win64 builds be commented out for now as #29 still applies?

Copy link
Member

Choose a reason for hiding this comment

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

I agree, let's comment this out and not distribute broken 64b builds for now

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Consider it done.

# asset_name: extract-xiso-win64-debug.zip
# asset_path: dist/extract-xiso-win64-debug.zip
# asset_content_type: application/zip
- name: Upload release assets (macOS build)
Copy link
Member

Choose a reason for hiding this comment

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

Let's also not distribute these 64b Mac builds until 64b is fixed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done, updated comments accordingly.

@mborgerson mborgerson merged commit 28a5f72 into XboxDev:master Dec 30, 2021
@mborgerson
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants