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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move related configuration together #4151

Merged
merged 2 commits into from
Sep 24, 2018
Merged

Conversation

deivid-rodriguez
Copy link
Contributor

@deivid-rodriguez deivid-rodriguez commented Sep 20, 2018

馃帺 What? Why?

This PR moves some related RSpec configuration bits together. The idea is that external plugins or applications can require these testing utilities and use them without needing to configure anything else.

馃搶 Related Issues

None.

馃搵 Subtasks

  • Add CHANGELOG entry
  • Add documentation regarding the feature
  • Add/modify seeds
  • Add tests
  • Another subtask

馃摲 Screenshots (optional)

None.

@ghost ghost added the status: WIP label Sep 20, 2018
So the files are self-contained and can be required in isolation.
@ghost ghost assigned mrcasals Sep 24, 2018
@ghost ghost added the status: WIP label Sep 24, 2018
@mrcasals mrcasals merged commit 7dfd99c into master Sep 24, 2018
@mrcasals mrcasals deleted the better_rspec_configuration branch September 24, 2018 07:55
@mrcasals
Copy link
Contributor

Wohoo! 馃挭

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.

None yet

2 participants