chore(flow): no constructor for class method needed

[florian-hoenicke]

Since load_config is a classmethod, we don't need to call it on a Flow object. Instead we can call the classmethod directly.

[codecov]

Codecov Report

Merging #1352 (c92fcda) into master (ac38ebf) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1352      +/-   ##
==========================================
+ Coverage   82.87%   82.93%   +0.05%     
==========================================
  Files         103      103              
  Lines        6746     6746              
==========================================
+ Hits         5591     5595       +4     
+ Misses       1155     1151       -4     

Impacted Files	Coverage Δ
jina/docker/hubio.py	64.85% <0.00%> (-0.86%)	⬇️
jina/peapods/pea.py	92.44% <0.00%> (+0.71%)	⬆️
jina/peapods/grpc_asyncio.py	80.61% <0.00%> (+5.10%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ac38ebf...c92fcda. Read the comment docs.

[bwanglzu]

Good catch!

[cristianmtr]

Nice, but then should't it be a staticmethod? I can't see that the cls is being used.

[JoanFM]

Nice, but then should't it be a staticmethod? I can't see that the cls is being used.

yes, u are right, it should be a static

[nan-wang]

@florian-hoenicke Good job! A minor comment. chore(xxx): will skip our unit tests. For the changes like this PR, it's better to use refactor(xxx):

[hanxiao]

@florian-hoenicke Good job! A minor comment. chore(xxx): will skip our unit tests. For the changes like this PR, it's better to use refactor(xxx):

correct: skip our cd not skip unit test

[florian-hoenicke]

@nan-wang Thanks, I will use refactor in the future.