-
Notifications
You must be signed in to change notification settings - Fork 0
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ntrnl 377 add environment variables to substitute reference to cabinet office or co words #64
Conversation
Coverage report
Test suite run success174 tests passing in 47 suites. Report generated by 🧪jest coverage report action from d9fbd01 |
On the
Organisation doesn't make sense in this context as we are civil servants are employed by departments. |
src/config/index.ts
Outdated
@@ -1,7 +1,8 @@ | |||
import { getEnvironmentValue } from '../utils/getEnvironmentValue'; | |||
|
|||
export const SERVICE_NAME = 'GitHub Requests'; | |||
export const ORG_NAME = 'The Cabinet Office'; | |||
export const DEPARTMENT_NAME = 'Cabinet Office'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These would probably be better in .env.example
so they are easier for users to change
src/config/index.ts
Outdated
@@ -1,7 +1,8 @@ | |||
import { getEnvironmentValue } from '../utils/getEnvironmentValue'; | |||
|
|||
export const SERVICE_NAME = 'GitHub Requests'; | |||
export const ORG_NAME = 'The Cabinet Office'; | |||
export const DEPARTMENT_NAME = 'Cabinet Office'; | |||
export const GITHUB_ORG_NAME = 'Cabinet Office'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be cabinetoffice
d5c8005
to
d9fbd01
Compare
@@ -10,6 +10,8 @@ GITHUB_OWNER="test" | |||
GITHUB_REPO_ISSUE_ASSIGNEE="test" | |||
GITHUB_REPO_ISSUE_LABEL="test" | |||
GITHUB_TERRAFORM_REPO="test" | |||
DEPARTMENT_NAME ="Cabinet Office" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
env vars are alphabetical
JIRA link
NTRNL-377
Description
Work checklist