Skip to content
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

⚙️ Adding dry-monads as development dependency #910

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

jeremyf
Copy link
Contributor

@jeremyf jeremyf commented Feb 7, 2024

Without the previously committed require 'dry-monads' we encounter the following error:

bundle exec rspec

NameError:
  uninitialized constant Dry::Monads::Result::Transformer
 ./spec/test_app/config/application.rb:15:in `<top (required)>'
 ./spec/test_app/config/environment.rb:4:in `require_relative'
 ./spec/test_app/config/environment.rb:4:in `<top (required)>'
 ./spec/rails_helper.rb:8:in `<top (required)>'
 ./spec/parsers/bulkrax/xml_parser_spec.rb:3:in `<top (required)>'

The error repeats everywhere.

When I add the development dependency and remove the require, I see the same error.

So, to be clear about this development dependency, I've added it to the gemspec.

See:

Without the previously committed `require 'dry-monads'` we encounter the
following error:

```
bundle exec rspec

NameError:
  uninitialized constant Dry::Monads::Result::Transformer
 ./spec/test_app/config/application.rb:15:in `<top (required)>'
 ./spec/test_app/config/environment.rb:4:in `require_relative'
 ./spec/test_app/config/environment.rb:4:in `<top (required)>'
 ./spec/rails_helper.rb:8:in `<top (required)>'
 ./spec/parsers/bulkrax/xml_parser_spec.rb:3:in `<top (required)>'
```

The error repeats everywhere.

When I add the development dependency and remove the require, I see the
same error.

So, to be clear about this development dependency, I've added it to the
gemspec.

See:

- #900 (comment)
- #900
@jeremyf jeremyf requested a review from orangewolf February 7, 2024 14:24
@jeremyf jeremyf added the ignore-for-release ignore this for release notes label Feb 7, 2024
@jeremyf jeremyf merged commit c49833f into main Feb 7, 2024
6 of 7 checks passed
@jeremyf jeremyf deleted the adding-dry-monads-dev-dependency branch February 7, 2024 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-for-release ignore this for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants