apache / airavata-mft

Apache Airavata Managed File Transfer Services
https://airavata.apache.org
Apache License 2.0
32 stars 39 forks source link

GCS Transfer python CLI integration #68

Closed Jayancv closed 1 year ago

Jayancv commented 1 year ago

@DImuthuUpe Service account creation steps added.

DImuthuUpe commented 1 year ago

@Jayancv thanks for the PR and it looks great. Before going further into the implementation, we might have to fix some minor formatting issues. I think your IDE auto-optimizes the code and you might have to update the schema or manually apply the right formatting. (I prefer the latter as it improves your coding discipline). I will point to few examples and please apply them to the rest of the code

DImuthuUpe commented 1 year ago

@Jayancv I have added some comments. Also, there is a merge conflict for some reason. Can you check?