Skip to content

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

Merged
onur-ozkan merged 1 commit intodevfrom
fix-ci-builds
Jul 5, 2024
Merged

fix(ci-builds): use ssh-deploy action for all builds except windows#2157
onur-ozkan merged 1 commit intodevfrom
fix-ci-builds

Conversation

@shamardy
Copy link
Copy Markdown
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
Copy Markdown

@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
@onur-ozkan onur-ozkan deleted the fix-ci-builds branch July 5, 2024 06:41
dimxy pushed 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