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

fix(ci-builds): use ssh-deploy action for all builds except windows #2157

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

shamardy
Copy link
Collaborator

@shamardy shamardy commented Jul 5, 2024

github-action-scp stopped working for all machines except windows for some reason, this PR switches to easingthemes/ssh-deploy@v5.0.3 for all builds uploads except windows ones. ssh-deploy uses rsync over ssh and brings some benefits such as using Node.js 20 in v5.0.3, data integrity of transferred files by using checksums and compressing file data during transfer among other things.

Copy link
Member

@onur-ozkan onur-ozkan left a comment

Choose a reason for hiding this comment

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

It's sad there is no new version for garygrossgarten/github-action-scp to bump... We should switch Windows ones too as soon as possible because this seems like an outdated plugin.

Merging this to remove development roadblock.

@onur-ozkan onur-ozkan merged commit acc4380 into dev Jul 5, 2024
23 of 26 checks passed
@onur-ozkan onur-ozkan deleted the fix-ci-builds branch July 5, 2024 06:41
dimxy added a commit that referenced this pull request Jul 21, 2024
* dev:
  feat(nft-swap): add standalone maker contract and proxy support (#2100)
  feat(ETH): add `gas_limit` coins param to override default values (#2137)
  feat(tendermint): implement better sequence resolving logic (#2164)
  ci(artifact): add target for macos on apple silicon (#2163)
  fix(helpers): extend http to ws address conversion (#2166)
  fix(makerbot): add "testcoin" to provider options (#2161)
  fix(hd_wallet): make extended pubkey of hd wallet generic (#2159)
  fix(docker-tests): implement containers runtime directories (#2162)
  feat(tendermint): improve the `max` handling for tendermint withdraw (#2155)
  revert #2158 (comment) (#2160)
  ci(artifacts): upload build artifacts with in-tree script (#2158)
  test(tendermint): migrate to local/offline containerized testnets (#2128)
  use easingthemes/ssh-deploy@v5.0.3 for all builds except windows (#2157)
  chore(bin): rename mm2 binaries to kdf (#2126)
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.

2 participants