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

Refactor gemspecs to no longer use gemspec_helper.rb. #78

Open
wants to merge 1 commit into
base: myron/dependabot-fixes/step3
Choose a base branch
from

Conversation

myronmarston
Copy link
Collaborator

Instead, inline the full definition of the gemspec in each .gemspec file. This should fix a dependabot error we are getting:

Bundler::GemspecError with message: [!] There was an error while loading `elasticgraph.gemspec`: uninitialized constant ElasticGraphGemspecHelper. Bundler cannot continue.

 #  from /home/dependabot/dependabot-updater/repo/elasticgraph/elasticgraph/elasticgraph.gemspec:14
 #  -------------------------------------------
 #
 >  ElasticGraphGemspecHelper.define_elasticgraph_gem(gemspec_file: __FILE__, category: :core) do |spec, eg_version|
 #    spec.summary = "ElasticGraph meta-gem that pulls in all the core ElasticGraph gems."
 #  -------------------------------------------

Instead, inline the full definition of the gemspec in each `.gemspec` file.
This should fix a dependabot error we are getting:

```
Bundler::GemspecError with message: [!] There was an error while loading `elasticgraph.gemspec`: uninitialized constant ElasticGraphGemspecHelper. Bundler cannot continue.

 #  from /home/dependabot/dependabot-updater/repo/elasticgraph/elasticgraph/elasticgraph.gemspec:14
 #  -------------------------------------------
 #
 >  ElasticGraphGemspecHelper.define_elasticgraph_gem(gemspec_file: __FILE__, category: :core) do |spec, eg_version|
 #    spec.summary = "ElasticGraph meta-gem that pulls in all the core ElasticGraph gems."
 #  -------------------------------------------
```
@myronmarston myronmarston force-pushed the myron/dependabot-fixes/step4 branch from 2f9c556 to db71460 Compare December 28, 2024 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant