replace commands that use Run to use RunE

[bhavyaKhatri2703]

fixes #675
replaced older commands from Run to RunE with errors being returned properly instead of jusing log.

[rshdhere]

@bhavyaKhatri2703 I see your commit messages. it's looking good, but follow contributing.md which has proper format for commit messages

ig dont need for this PR but do consider it in the upcoming PRs

[rshdhere]

here's a reference for you, from the official contributing md file. you dont have to force-push it again if you follow this

[bhavyaKhatri2703]

understood thanks for sharing @rshdhere

[bhavyaKhatri2703]

can someone help me with these checks?

[codecov]

Codecov Report

❌ Patch coverage is 1.77515% with 166 lines in your changes missing coverage. Please review.
✅ Project coverage is 7.25%. Comparing base (60ad0bd) to head (76761fc).
⚠️ Report is 95 commits behind head on main.

Files with missing lines	Patch %	Lines
cmd/harbor/root/tag/immutable/list.go	0.00%	15 Missing ⚠️
cmd/harbor/root/logs.go	0.00%	11 Missing ⚠️
cmd/harbor/root/user/delete.go	0.00%	10 Missing ⚠️
cmd/harbor/root/project/robot/refresh.go	0.00%	8 Missing ⚠️
cmd/harbor/root/project/robot/update.go	0.00%	8 Missing ⚠️
cmd/harbor/root/user/elevate.go	0.00%	8 Missing ⚠️
cmd/harbor/root/robot/refresh.go	0.00%	7 Missing ⚠️
cmd/harbor/root/context/switch.go	0.00%	5 Missing and 1 partial ⚠️
cmd/harbor/root/project/robot/view.go	0.00%	6 Missing ⚠️
cmd/harbor/root/quota/list.go	0.00%	6 Missing ⚠️
... and 20 more

Additional details and impacted files

@@            Coverage Diff            @@
##             main    #679      +/-   ##
=========================================
- Coverage   10.99%   7.25%   -3.74%     
=========================================
  Files         173     260      +87     
  Lines        8671   12909    +4238     
=========================================
- Hits          953     936      -17     
- Misses       7612   11865    +4253     
- Partials      106     108       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[NucleoFusion]

So right now, lint is failing. Which probably will be fixed if you run this

dagger call run-doc export --path=./doc

[NucleoFusion]

Also, could you remove changes to the artifact directory? It seems to have some different format errors as well.
So It would be better if I address the refactor and the RunE usage in a seperate PR (that I will create). and not have any merge conflicts here

[bhavyaKhatri2703]

Also, could you remove changes to the artifact directory? It seems to have some different format errors as well. So It would be better if I address the refactor and the RunE usage in a seperate PR (that I will create). and not have any merge conflicts here

done

[bhavyaKhatri2703]

So right now, lint is failing. Which probably will be fixed if you run this

dagger call run-doc export --path=./doc

done

[NucleoFusion]

lgtm!

[bupd]

@bhavyaKhatri2703 please resolve conflicts

Thanks for the contribution.

[bhavyaKhatri2703]

@bhavyaKhatri2703 please resolve conflicts

Thanks for the contribution.

@bupd resolved, thanks

I think lint is failing maybe because of issues in main