Skip to content

Commit

Permalink
shoryuken: Add types for Shoryuken.logger (#432)
Browse files Browse the repository at this point in the history
  • Loading branch information
tk0miya authored Oct 23, 2023
1 parent c65a7f1 commit c729dfd
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 8 deletions.
2 changes: 1 addition & 1 deletion gems/shoryuken/6.0/_scripts/test
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ RBS_DIR=$(cd $(dirname $0)/..; pwd)
# Set REPO_DIR variable to validate RBS files added to the corresponding folder
REPO_DIR=$(cd $(dirname $0)/../../..; pwd)
# Validate RBS files, using the bundler environment present
bundle exec rbs --repo $REPO_DIR -r shoryuken:6.0 validate --silent
bundle exec rbs --repo $REPO_DIR -r shoryuken:6.0 -r logger validate --silent

cd ${RBS_DIR}/_test
# Run type checks
Expand Down
2 changes: 2 additions & 0 deletions gems/shoryuken/6.0/_test/Steepfile
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,7 @@ target :test do
repo_path "../../../"
library "shoryuken"

library "logger"

configure_code_diagnostics(D::Ruby.all_error)
end
9 changes: 2 additions & 7 deletions gems/shoryuken/6.0/manifest.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,2 @@
# manifest.yaml describes dependencies which do not appear in the gemspec.
# If this gem includes such dependencies, comment-out the following lines and
# declare the dependencies.
# If all dependencies appear in the gemspec, you should remove this file.
#
# dependencies:
# - name: pathname
dependencies:
- name: logger
2 changes: 2 additions & 0 deletions gems/shoryuken/6.0/shoryuken.rbs
Original file line number Diff line number Diff line change
Expand Up @@ -17,4 +17,6 @@ module Shoryuken
def shoryuken_options: (?queue: String, ?delete: bool, ?auto_delete: bool, ?auto_visibility_timeout: bool, ?retry_intervals: Numeric, ?batch: bool) -> void
end
end

def self.logger: () -> Logger
end

0 comments on commit c729dfd

Please sign in to comment.