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 2.09790% with 140 lines in your changes missing coverage. Please review.
✅ Project coverage is 7.80%. Comparing base (60ad0bd) to head (a6c14ed).
⚠️ Report is 119 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/project/robot/update.go	0.00%	10 Missing ⚠️
cmd/harbor/root/project/robot/refresh.go	0.00%	8 Missing ⚠️
cmd/harbor/root/project/robot/view.go	0.00%	8 Missing ⚠️
cmd/harbor/root/user/delete.go	0.00%	7 Missing ⚠️
cmd/harbor/root/user/password.go	0.00%	6 Missing and 1 partial ⚠️
cmd/harbor/root/context/switch.go	0.00%	5 Missing and 1 partial ⚠️
cmd/harbor/root/quota/list.go	0.00%	6 Missing ⚠️
cmd/harbor/root/quota/update.go	0.00%	6 Missing ⚠️
cmd/harbor/root/repository/view.go	0.00%	6 Missing ⚠️
... and 16 more

Additional details and impacted files

@@            Coverage Diff            @@
##             main    #679      +/-   ##
=========================================
- Coverage   10.99%   7.80%   -3.19%     
=========================================
  Files         173     270      +97     
  Lines        8671   13166    +4495     
=========================================
+ Hits          953    1028      +75     
- Misses       7612   12025    +4413     
- Partials      106     113       +7     

☔ 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

[bupd]

@bhavyaKhatri2703 lint issue in main is fixed - please resolve conflicts

Thanks for the contribution

[bupd]

please rebase onto main

Thanks

[bhavyaKhatri2703]

@bupd pls have a look now , thanks.

[bupd]

Looks like irrelevant doc changes have been made please fix that

Thanks

[bhavyaKhatri2703]

can someone help me with docs generation?
when i generate docs via dagger call run-doc export --path=./doc
it introduces these whitespace changes d8e4fe0

When i try to revert them back manually lints get failed.

[NucleoFusion]

@bhavyaKhatri2703 let the doc changes be, just run run-doc and push it.
run-doc is supposed to change to tell the lint checker that docs have been changed

[bhavyaKhatri2703]

@bupd let me know if there are still unwanted changes?

[bupd]

Please fix the ci - test-code pipeline is failing

otherwise LGTM

@bhavyaKhatri2703 Thanks for the contribution

[bhavyaKhatri2703]

Please fix the ci - test-code pipeline is failing

otherwise LGTM

@bhavyaKhatri2703 Thanks for the contribution

fixed now.

[qcserestipy]

LGTM