Skip to content

Commit

Permalink
Missing require (#206)
Browse files Browse the repository at this point in the history
* Fix missing version require not triggering ci failure

bundle exec loads gem/version through the gemspec file
CI doesn't need bundle exec since each gem will only be available once anyways

Signed-off-by: Earlopain <[email protected]>
  • Loading branch information
Earlopain authored Sep 18, 2023
1 parent a7cb0fb commit 503ad3c
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 6 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/compatibility.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
ruby -v
bundle install
- name: Run All Tests
run: bundle exec rake test:all
run: rake test:all

test-opensearch-security:
env:
Expand Down Expand Up @@ -77,4 +77,4 @@ jobs:
ruby -v
bundle install
- name: opensearch-ruby
run: bundle exec rake test:client:security
run: rake test:client:security
6 changes: 3 additions & 3 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
ruby -v
bundle install
- name: Run All Tests
run: bundle exec rake test:all
run: rake test:all

test-opensearch-faraday-1:
env:
Expand Down Expand Up @@ -78,7 +78,7 @@ jobs:
ruby -v
bundle install
- name: Test Faraday on Transport Layer
run: bundle exec rake test:transport:all
run: rake test:transport:all

test-opensearch-security:
env:
Expand Down Expand Up @@ -112,4 +112,4 @@ jobs:
ruby -v
bundle install
- name: Test Client Security
run: bundle exec rake test:client:security
run: rake test:client:security
2 changes: 1 addition & 1 deletion .github/workflows/test-unreleased.yml
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ jobs:

- name: opensearch-ruby
working-directory: ruby-client
run: bundle exec rake test:integration
run: rake test:integration

- name: Save server logs
if: failure()
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
### Deprecated
### Removed
### Fixed
- Fixed missing version require [#205](https://github.com/opensearch-project/opensearch-ruby/issues/205) ([#206](https://github.com/opensearch-project/opensearch-ruby/pull/206))
### Security

## [3.0.0]
Expand Down
1 change: 1 addition & 0 deletions lib/opensearch.rb
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
# specific language governing permissions and limitations
# under the License.

require 'opensearch/version'
require 'opensearch/transport'
require 'opensearch/api'

Expand Down

0 comments on commit 503ad3c

Please sign in to comment.