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

[pull] main from bkeepers:main #1

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

[pull] main from bkeepers:main #1

wants to merge 19 commits into from

Conversation

pull[bot]
Copy link

@pull pull bot commented Jun 2, 2024

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

mikegee and others added 2 commits May 20, 2024 14:40
I think this was a typo.
Spell out 'FILENAME' in readme
@pull pull bot added the ⤵️ pull label Jun 2, 2024
bkeepers and others added 11 commits June 6, 2024 11:18
By providing a 'changelog_uri' in the metadata of the gemspec a
'Changelog' link will be shown on https://rubygems.org/gems/dotenv
which makes it quick and easy for someone to check on the changes
introduced with a new version.

Details of this functionality can be found on https://guides.rubygems.org/specification-reference/
…_rubygems

Provide a 'Changelog' link on rubygems.org/gems/dotenv
This makes it possible for the user to selectively use Spring on a command to command basis

Fixes #510
Only use Spring if already loaded
Copy link

changeset-bot bot commented Sep 17, 2024

⚠️ No Changeset found

Latest commit: ee3975c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

bkeepers and others added 6 commits September 17, 2024 12:03
An exception occurs when invoking the rake task via ./bin/rails.

```
bin/rails aborted!
NoMethodError: undefined method `watch' for Spring:Module (NoMethodError)

    Spring.watch event.payload[:env].filename if Rails.application
          ^^^^^^
.bundle/ruby/3.2.0/gems/dotenv-3.1.3/lib/dotenv/rails.rb:16:in `block in <main>'
```

When using `./bin/rails`, 'spring/client' is loaded, but 'spring/watcher', which is unnecessary for the client-side, is not required.

This PR follows up on #511 and fixes exception.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants