Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat:
dnf
module #377base: main
Are you sure you want to change the base?
feat:
dnf
module #377Changes from 18 commits
fd4ad28
cd91b05
cf3d495
2f392e7
6d1047f
9d13c87
1396a18
b898af2
6523979
97566b8
0bb79d0
70c5766
e01ed40
7d4aff9
a4e9ba4
7da24f5
d495b6a
1137c8b
eb522e3
57f8b22
7a5bcea
c7ae430
b1a7c6f
b7c0206
feb0d71
fba3840
e27ba39
a53dd5b
1a81609
a72ae5c
2790123
ecffeae
84f89d0
5d7f34b
7fc5a24
63f745b
b474a5a
770bb33
d8a9006
33065b4
3423190
f176120
7458b4b
ce4163c
c8043a4
aa92eff
8265b85
c9f4fb0
92df6a0
e66b176
4920cc4
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[shellcheck] reported by reviewdog 🐶
$/$ {} is unnecessary on arithmetic variables. SC2004
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this can get confusing for users. This will be seen as an object and not a string. We could have a separate property
copr
that will take an array of strings which we could then iterate over and feed into thednf copr enable
command.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks less clean & more separated if it's done this way, hence why I went with
copr: user/project
format.Is
:
making it an object or the whitespace?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can maybe have
COPR user/project
as format insteadThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed it to
COPR user/project
formatThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still don't think this is better. Having a
copr:
property and a list of COPR repos would be very clear. Encoding meaning into the spec will be easier for a user to understand how to add new COPR repos and lead to fewer mistakes like accidentally having too much whitespace betweenCOPR
and the repo path.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📝 [shellcheck] reported by reviewdog 🐶
Possible misspelling: REPOS may not be assigned. Did you mean REPO? SC2153
modules/modules/rpm-ostree/rpm-ostree.sh
Line 8 in 1137c8b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[shellcheck] reported by reviewdog 🐶
$/$ {} is unnecessary on arithmetic variables. SC2004
modules/modules/rpm-ostree/rpm-ostree.sh
Line 72 in 1137c8b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quote this to prevent word splitting. SC2046
modules/modules/rpm-ostree/rpm-ostree.sh
Line 115 in 1137c8b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quote this to prevent word splitting. SC2046
modules/modules/rpm-ostree/rpm-ostree.sh
Line 117 in 1137c8b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quote this to prevent word splitting. SC2046
modules/modules/rpm-ostree/rpm-ostree.sh
Line 120 in 1137c8b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quote this to prevent word splitting. SC2046
modules/modules/rpm-ostree/rpm-ostree.sh
Line 123 in 1137c8b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📝 [shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. SC2086
modules/modules/rpm-ostree/rpm-ostree.sh
Line 185 in 1137c8b