fix(ci): added pragma to dagger functions to pass a default directory to the source flag#699
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the Dagger pipeline entrypoints so the source *dagger.Directory argument defaults to the repository root (and respects .gitignore), making --source no longer necessary for common dagger call invocations (per #696).
Changes:
- Added Dagger parameter pragmas
// +ignore=[".gitignore"]and// +defaultPath="."tosourcearguments. - Applied the change across the lint, test, vuln-check, docs, and dev-build Dagger functions.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
.dagger/vuln.go |
Adds default source directory + ignore rules to vulnerability check functions. |
.dagger/test.go |
Adds default source directory + ignore rules to test and coverage functions. |
.dagger/lint.go |
Adds default source directory + ignore rules to lint functions. |
.dagger/doc.go |
Adds default source directory + ignore rules to doc generation function. |
.dagger/buildDev.go |
Adds default source directory + ignore rules to the dev build function. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@bupd @NucleoFusion can we merge this? |
qcserestipy
left a comment
There was a problem hiding this comment.
@rkcoder101 please fix the merge conflicts with upstream main
…he source flag Signed-off-by: Rayyan Khan <rayyanrehman101@gmail.com>
d2f8ad6 to
58fa0f4
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #699 +/- ##
=========================================
- Coverage 10.99% 7.23% -3.76%
=========================================
Files 173 261 +88
Lines 8671 12923 +4252
=========================================
- Hits 953 935 -18
- Misses 7612 11880 +4268
- Partials 106 108 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
--sourceoptional in Dagger functions #696