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.

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

add voting for councils tests #210

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

add voting for councils tests #210

wants to merge 2 commits into from

Conversation

RKatarine
Copy link
Contributor

No description provided.

@@ -5,7 +5,7 @@ import yn from 'yn';
import type {PlaywrightTestConfig} from '@playwright/test';
import {devices} from '@playwright/test';

const dotenvFiles = ['.env.local', '.env'];
const dotenvFiles = ['.env.local.example', '.env'];
Copy link
Contributor

@extg extg Nov 8, 2022

Choose a reason for hiding this comment

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

Не надо так делать, пожалуйста. Экзампл на то и экзампл, значения из него не должны где либо использоваться. Если хочется чтобы значения из экзампла автоматически скопировали в .env.local, то можно сделать так

// package.json
  "scripts": {
    "postinstall": "[[ ! -f .env.local ]]  && cp .env.local.example .env.local || true && yarn generate:api",

Тогда после первой установки зависимостей из .env.local.example все значения скопируются в .env.local, при этом при последующих установках зависимостей значения которые возможно каждый менял в своем личном (потом что он не под гитом) .env.local сохранятся. Т.е. копирование произойдет только в случае, если в директории проекта еще нет .env.local

Copy link
Contributor

Choose a reason for hiding this comment

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

но тут получился сложный скрипт, который тяжело читается, поэтому предлагаю создать файл .scripts/postinstall.sh в который положить ровно тоже самое

#!/usr/bin/env bash

LOCAL_DOTENV_PATH=".env.local"

# Api should be generated in CI and locally
yarn generate:api

if [[ ! -z "$CI" ]]; then
  echo "${YELLOW}Skipping postinstall"
  exit 0
fi

# .env.local should be copied only locally
if [[ ! -f "$LOCAL_DOTENV_PATH" ]]; then
  echo "Copying $LOCAL_DOTENV_PATH.example to $LOCAL_DOTENV_PATH"
  cp "$LOCAL_DOTENV_PATH.example" "$LOCAL_DOTENV_PATH"
fi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants