-
Notifications
You must be signed in to change notification settings - Fork 2
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
updates for OS 3.8 #5
base: master
Are you sure you want to change the base?
Conversation
spec.add_dependency 'openstudio-extension', '~> 0.8.0' | ||
spec.add_dependency 'openstudio-standards', '0.6.1' | ||
|
||
spec.add_development_dependency 'bundler', '~> 2.4.10' | ||
spec.add_development_dependency 'rake', '~> 13.0' |
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 we can also remove the requirements for rake
and bundler
for the same reason as above: they are included with extension-gem. In fact, I think we should remove them to prevent accidentally trying to use an incorrect version.
@@ -22,11 +22,14 @@ Gem::Specification.new do |spec| | |||
spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) } | |||
spec.require_paths = ['lib'] | |||
|
|||
spec.add_development_dependency 'bundler', '~> 2.1' | |||
spec.required_ruby_version = '~> 3.2.2' |
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.
spec.required_ruby_version = '~> 3.2.2' | |
spec.required_ruby_version = '3.2.2' |
We support exactly Ruby v3.2.2 because os-extension-gem requires bundler==2.4.10 and that requires Ruby 3.2.2: https://stdgems.org/bundler/
@KaiyuSun Have you had a chance to work on this? We'd love to use this with OS 3.8. |
…exist problem in ruby 3
…pgrading to OS 3.8
Upgraded to OpenStudio 3.8 and Ruby 3.2.2. All measures have been tested through. |
@KaiyuSun, I started this PR to help with the upgrades for OpenStudio 3.8 and Ruby 3.2.2. I didn't test any of the measures with OS 3.8 though. Let me know if you have any questions or run into issues with the measure updates!