parse-community / parse-community/parse-server
ci(build): run a build in non-dev environment
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 21.4k
- Forks
- 4.8k
- Avg merge
- 7h 45m
- Merged PRs (30d)
- 11
Description
New Feature / Enhancement Checklist
- I am not disclosing a vulnerability.
- I am not just asking a question.
- I have searched through existing issues.
Current Limitation
Currently, the CI runs all tests in dev environments. This means all npm dev dependencies will be installed. That is not representative of a production environment, where dev dependencies are not installed.
This can cause bugs to stay undetected, like https://github.com/parse-community/parse-server/pull/7183 where a dev dependency is used in non-test code and therefore passes the test. In a production environment that dependency would be missing and break the feature that is using that dependency.
Feature / Enhancement Description
Add additional CI stage to run tests in non-dev environment.
Example Use Case
https://github.com/parse-community/parse-server/pull/7183
Alternatives / Workarounds
none
3rd Party References
n/a
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the repository's existing CI configuration and test commands, then determine how dependencies are installed for the current development environment. Add a non-development CI stage that runs the tests without npm development dependencies, and verify that the pipeline catches production-code imports unavailable in that environment.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, nodejs
- Domain
- ci-cd, testing
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100