Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@shreyas4510
Copy link
Contributor

Add an extra step to the setup process to include the creation of an env file. Update the documentation to reflect this new step for easier setup.

Closes #98

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Add an extra step to the setup process to include the creation of an env file.
Update the documentation to reflect this new step for easier setup.

Closes eicrud#98
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can remove the TEST_CRUD_DB since it is set in the commands

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need the cross env here, running npm run test runs mongoDB tests and npm run test:postgre postgreSQL tests

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we could add a test:mongo command to make it more clear

Updated the test command to support MongoDB-related tests and ensure compatibility
with the latest configurations. Additionally, updated the documentation to reflect
the changes made to the test command and included new branch naming conventions.
@acrosett acrosett merged commit e7ddec7 into eicrud:develop Nov 9, 2024
4 checks passed
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.

Add dynamic PostgreSQL configuration for setup

2 participants