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.

Sign up for GitHub

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

Jump to bottom

Add require factory_bot_rails to spec_helper to attempt to fix seed #96

Merged
merged 1 commit into from
Jan 7, 2021
Merged

Add require factory_bot_rails to spec_helper to attempt to fix seed #96

merged 1 commit into from
Jan 7, 2021

Conversation

LHJE
Copy link
Collaborator

@LHJE LHJE commented Jan 7, 2021 •
edited
Loading

Code Highlights:

  • Adds require factory_bot_rails to spec_helper to attempt to fix seeding Heroku

Where should the reviewer start?

  • spec_helper

Have Tests Been Added?

  • No.
  • Yes, but all tests are not passing.
  • Yes, and all are passing.

Any background context you want to provide?

Message/Questions for reviewer:

Issues:

Screenshots (if appropriate):

Tracking Consistency:

  • added appropriate labels
  • My code follows the code style of this project and has removed all unnecessary annotations
  • I have added comments on my pull request, particularly in hard-to-understand areas
  • looked at PR preview to check spelling, syntax, formatting, and completion
  • Rubocop Violations:

@LHJE LHJE requested a review from AngelaGuardia January 7, 2021 18:05
@LHJE LHJE self-assigned this Jan 7, 2021
@AngelaGuardia AngelaGuardia merged commit 2d75a29 into main Jan 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AngelaGuardia AngelaGuardia Awaiting requested review from AngelaGuardia

Assignees

@LHJE LHJE

Labels
BEE database
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Drop prod db, re-create, migrate and seed
2 participants
@LHJE @AngelaGuardia