Skip to content

Github workflow action for building & releasing Bitshares beet#192

Merged
sschiessl-bcp merged 4 commits intobitshares:developfrom
grctest:action
Sep 28, 2022
Merged

Github workflow action for building & releasing Bitshares beet#192
sschiessl-bcp merged 4 commits intobitshares:developfrom
grctest:action

Conversation

@grctest
Copy link
Contributor

@grctest grctest commented Sep 10, 2022

Still a WIP, the output setup results in the following error after launching the app post install: cannot find module '@babel/runtime/helpers/asyncToGenerator'

I've tried installing the @babel/runtime package but this doesn't change the error.

The workflow focuses on the windows build for now, however once fully operational the next priority is suppporting linux/macos.

@abitmore
Copy link
Member

Still a WIP, the output setup results in the following error after launching the app post install: cannot find module '@babel/runtime/helpers/asyncToGenerator'

I found this: yarnpkg/yarn#8054 (comment):

export NODE_PATH=/usr/lib/nodejs:/usr/share/nodejs

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=933229

babel runtime installed and added to allowlist
small babel config change
build script uses npm
@grctest
Copy link
Contributor Author

grctest commented Sep 12, 2022

Cool, that's github actions successfully building windows setup: https://github.com/grctest/beet/releases/tag/v1.0.3

@grctest
Copy link
Contributor Author

grctest commented Sep 12, 2022

Looks like appveyor still wants to run without the appveyor file?

@sschiessl-bcp
Copy link
Collaborator

Looks also like it is (sometimes?) working https://ci.appveyor.com/project/clockworkgr/beet/builds/44753286/artifacts

Possible we need to tun the link off from AppVeyor end too...

Additional os packages
@grctest
Copy link
Contributor Author

grctest commented Sep 13, 2022

As far as I can tell the action workflow has successfully built and packaged the linux deb + appimage, as well as macos dmg. https://github.com/grctest/beet/releases/tag/v1.0.9 (not tested running these new packages on their respective os yet)

@grctest
Copy link
Contributor Author

grctest commented Sep 13, 2022

RE uses: BTS-CM/action-gh-release@cd28b0f5ee8571b76cfdaa62a30d51d752317477 I forked the action-gh-release so as to freeze the state of the workflow action to that commit, reducing risk of supply chain attack.

@grctest
Copy link
Contributor Author

grctest commented Sep 14, 2022

I've successfully implemented virustotal scanning of deliverables in another repo, should we add this to beet too? In order to establish presence as a safe application in all/most antivirus apps. https://github.com/BTS-CM/NFT_Viewer/releases/tag/v1.0.26 https://github.com/BTS-CM/NFT_Viewer/blob/main/.github/workflows/node.js.yml https://hive.blog/bitshares/@nftea.gallery/automatically-scanning-release-binaries-with-over-70-antiviruses-via-virustotal-in-github-action-workflows

@sschiessl-bcp
Copy link
Collaborator

Did you see the changelist actions https://github.com/bitshares/bitshares-ui/blob/c0a2774f8703b23743c97fe159f19f946770e497/.github/workflows/build-release-binaries.yml#L43, and what is the reason to use a custom release actions instead of actions/create-release?

@grctest
Copy link
Contributor Author

grctest commented Sep 14, 2022

what is the reason to use a custom release actions instead of actions/create-release

actions/create-release is unmaintained & archived, they recommended https://github.com/softprops/action-gh-release which I then forked to freeze our use to a specific commit to reduce risk of potential supply chain attack, perhaps we can use the softprops with a specific commit rather than fork the repo

@grctest
Copy link
Contributor Author

grctest commented Sep 14, 2022

Did you see the changelist actions

Do you want to use that action repo for changelogs? What do you think of this action instead? https://github.com/marketplace/actions/release-changelog-builder

Avoid adding duplicate accounts in the same wallet
Fix wallet name bug during creation
Avoid using text component
@grctest
Copy link
Contributor Author

grctest commented Sep 23, 2022

@sschiessl-bcp Any further thoughts on the above?

@sschiessl-bcp
Copy link
Collaborator

@sschiessl-bcp Any further thoughts on the above?

Not at the moment

@sschiessl-bcp sschiessl-bcp merged commit 9397463 into bitshares:develop Sep 28, 2022
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