-
Notifications
You must be signed in to change notification settings - Fork 234
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
Bump audited gem to latest #4080
Bump audited gem to latest #4080
Conversation
@@ -247,7 +247,7 @@ def assert_content(files) | |||
next if association.name == :audits # should never be destroyed | |||
next if association.options.key?(:through) # already cleaned up via through relation | |||
next if association.options.key?(:dependent) # already defined | |||
next if association.is_a?(Doorkeeper::Application) && association.name == :authorized_tokens | |||
next if association.name == :authorized_tokens # removed only when user is deleted; otherwise soft deleted |
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.
seems a bit sus ... but if it works 🤷
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.
This was the exception we talked about for the Doorkeeper gem bump merged in yesterday. Not sure why the specs decided to grump about the syntax today, rather than yesterday when I was actually working on this 😅
Description
Striving to catch up on getting gems bumped to latest versions, this PR aims to get the
audited
gem up to latest.Risks