Generalize basic CI script #127
Conversation
| echo "CORE_PATCH_NAME=$patch_name" >> $GITHUB_ENV | ||
|
|
||
| # we can get number, otherwise set up master | ||
| vers_number=$(echo "$patch_name"|tr -d -c 0-9) |
There was a problem hiding this comment.
предлагается улучшение:
тут vers_number будет пустым, если название патча было, например: aqo_master.patch.
Можно без дополнительной проверки сразу добавить блок на else:
vers_number="$(echo "$patch_name"|tr -d -c 0-9)"'master'
тогда vers_number сразу будет определено и ниже дополнительная проверка не нужна
.github/workflows/c-cpp.yml
Outdated
|
|
||
| - name: "Prepare PG directory" | ||
| run: | | ||
| git clone https://github.com/postgres/postgres.git $GITHUB_WORKSPACE/../pg |
There was a problem hiding this comment.
думаю можно ограничиться клонированием не всего репозитория, а определенной ветки, на которой будет тестироваться AQO (сэкономим память). То есть:
git clone -b $branch_name --depth $depth --single-branch https://github.com/postgres/postgres.git $GITHUB_WORKSPACE/../pg
| cd $GITHUB_WORKSPACE/../pg | ||
| git checkout $PG_BRANCH | ||
| ls -la | ||
| cp -r ../aqo contrib/aqo |
There was a problem hiding this comment.
не обязательно, но думаю, можно не копировать, а переместить файлы, так как у каждого раннера, скажем так свои файлы, и, если не перемещать,можно не копировать .git. Я использовала такую команду, если адаптировать ее под анный раннер:
mkdir contrib/aqo &&
mv $(find . -maxdepth 1 -mindepth 1 -not -name ".git") contrib/aqo
danolivo
force-pushed
the
ci-generalization
branch
2 times, most recently
from
c80dfe3 to
de61cab
Compare
reviewed-by: a.rybakina
danolivo
force-pushed
the
ci-generalization
branch
from
de61cab to
43efdb5
Compare
No description provided.