search

aergoio / aergo

aergo blockchain kernel
http://docs.aergo.io
MIT License
214 stars 44 forks source link

tidy packages #325

Closed rabbitprincess closed 9 months ago

rabbitprincess commented 9 months ago

What is Changed?

remove unnecessary files - dockerfile, travis move to tools - example, polaris move to types - message

Is there risk of fork?

no

kroggen commented 9 months ago

There is code on the travis.ci that is not moved to another place, mainly the generation of release packages (see on the before_deploy section)

The last releases do not include binaries

At least we can access the file via this PR if we need it later

kroggen commented 9 months ago

Why polaris was not moved to cmd? There is already a folder with that name there What is the difference between them?

kroggen commented 9 months ago

  1. If aergosvr uses polaris client, then should we consider it an external tool?

  2. Why there is polaris in cmd and in tool?

rabbitprincess commented 9 months ago

If aergosvr uses polaris client, then it is not an external tool, right?

i agree. it seems good to move in cmd, not tools.