citusdata / citusdata/the-process

Initial status value

Open
#21 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Dockerfile
Stars
6
Forks
7
Avg merge
4d 22h
Merged PRs (30d)
1

Description

Currently we have a structure like this in our scripts:
```
status = 0
# run the test and assign the exit code to status
exit(status)
```

Wouldnt it be better to start with a nonzero status code so that if we dont run our tests the script will return with a nonzero code? Currently if the tests are not run the initial status which is `0` will be returned.

An example can be seen here:
https://github.com/citusdata/the-process/blob/upgradeTester/circleci/images/exttester/files/sbin/install-and-test-ext#L15

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.