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

chore(env): update env files & scripts package json #708

Closed
wants to merge 12 commits into from
Closed

chore(env): update env files & scripts package json #708

wants to merge 12 commits into from

Conversation

rebeccadumazert
Copy link
Contributor

@rebeccadumazert rebeccadumazert requested review from douglasduteil and rdubigny and removed request for douglasduteil September 25, 2024 10:46
douglasduteil
douglasduteil approved these changes Sep 25, 2024
View reviewed changes
Copy link
Member

@douglasduteil douglasduteil left a comment

Choose a reason for hiding this comment

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

@douglasduteil
Copy link
Member

🔧 Alignement des règles TypeScript

@rdubigny rdubigny changed the title chore(env): updtate env files & scripts package json chore(env): update env files & scripts package json Sep 25, 2024
douglasduteil
douglasduteil reviewed Sep 25, 2024
View reviewed changes
.env.test Outdated
@@ -1,3 +1,4 @@
CONSIDER_ALL_EMAIL_DOMAINS_AS_NON_FREE=False
DO_NOT_USE_ANNUAIRE_EMAILS=False
DO_NOT_CHECK_EMAIL_DELIVERABILITY=False
DATABASE_URL=postgres://moncomptepro:[email protected]:5432/moncomptepro
Copy link
Member

Choose a reason for hiding this comment

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

@rdubigny This is temporally require to make "npm test" happy as it's importing the global config/env.ts

@douglasduteil douglasduteil mentioned this pull request Oct 3, 2024
Merged
@rdubigny
Copy link
Contributor

duplicates #708

@rdubigny rdubigny closed this Oct 15, 2024
@rdubigny rdubigny deleted the RDUM-update-env branch October 15, 2024 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@douglasduteil douglasduteil douglasduteil approved these changes

@rdubigny rdubigny rdubigny approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rebeccadumazert @douglasduteil @rdubigny