-
-
Notifications
You must be signed in to change notification settings - Fork 927
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
Build trusted publisher filter #5237
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -411,6 +411,10 @@ def gem_file_name | |
"#{full_name}.gem" | ||
end | ||
|
||
def pushed_by_trusted_publisher? | ||
pusher_api_key&.trusted_publisher? ? true : false | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Doesn't There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The safe navigator means a nil value can be returned if the version has no There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Would There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No, but There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Or maybe There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Am I the only one that prefers |
||
end | ||
|
||
private | ||
|
||
def update_prerelease | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -844,6 +844,7 @@ de: | |
summary: | ||
description: | ||
downloads: | ||
trusted_publisher: | ||
updated: | ||
yanked: | ||
show: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -748,6 +748,7 @@ nl: | |
summary: | ||
description: | ||
downloads: | ||
trusted_publisher: | ||
updated: | ||
yanked: | ||
show: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -772,6 +772,7 @@ pt-BR: | |
summary: | ||
description: | ||
downloads: | ||
trusted_publisher: | ||
updated: | ||
yanked: | ||
show: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -760,6 +760,7 @@ zh-CN: | |
summary: 概要 | ||
description: 描述 | ||
downloads: 下载数 | ||
trusted_publisher: | ||
updated: 更新 | ||
yanked: 撤回 | ||
show: | ||
|
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.
Does this mapping change need full re-indexing?
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.
Good question. I would not think so, since there is no change to the database source schema, but I am not sure. It is my first time committing to this repo and I do not have full context, so I will wait for others to weigh in.
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.
Any new mappings do require a reindex, which is easy to do but needs some coordination when deployed.