-
Notifications
You must be signed in to change notification settings - Fork 0
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
Ie 516 Update to be compatible with Ruby 3 #22
Conversation
ncperry
commented
Feb 1, 2024
•
edited
Loading
edited
- Added CI step
- Removed JSON as a gem. By using the system-level JSON module, instead of the JSON gem, we'll be guaranteed to have a version of the JSON library that is compatible with the ruby compiler.
- Updated tz-info gem as the older gem had a problem with ruby 3 style keyword arguments
- Updated rspec to be compatible with RubyMine
- Fixed a few tests that had been broken in previous PRs but that weren't caught
using the older gem implementation of json was introducing a arity error because of ruby 3 keywords
In #19 we changed the default version but we never updated the specs accordingly.
@@ -56,7 +56,7 @@ | |||
|
|||
context 'when the wsdl has not been set' do | |||
it 'returns a path to the WSDL to use for the API' do | |||
expect(config.wsdl).to eq("https://webservices.netsuite.com/wsdl/v2015_1_0/netsuite.wsdl") | |||
expect(config.wsdl).to eq("https://webservices.netsuite.com/wsdl/v2023_1_0/netsuite.wsdl") |
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 am really curious why this was changed originally
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.
LGTM, way to leave it better than you found it
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.
🥇 Clean and meaningful changes