Skip to content

Move ~/runner code to ~/ci/runner

Miguel Rincon requested to merge mrincon-runner-ci-frontend-move into master

What does this MR do and why?

Move ~/runner code to ~/ci/runner

This change moves the runner directories to use the /ci prefix so we can assign CODEOWNERS to the /ci directory.

Commands and replacements used

# Pre-replacement commands:

mkdir app/assets/javascripts/ci/
mv app/assets/javascripts/runner/ app/assets/javascripts/ci/
mkdir spec/frontend/ci/
mv spec/frontend/runner/ spec/frontend/ci/
mkdir ee/app/assets/javascripts/ci/
mv ee/app/assets/javascripts/runner/ ee/app/assets/javascripts/ci/
mkdir ee/spec/frontend/ci/
mv ee/spec/frontend/runner/ ee/spec/frontend/ci/
rm -rf graphql/ci/runner/
rm -rf tmp/tests/frontend/fixtures-ee/graphql/runner/

# Replacement patterns:

~/runner -> ~/ci/runner
ee/runner -> ee/ci/runner
ee_else_ce/runner -> ee_else_ce/ci/runner
ee_component/runner/ -> ee_component/ci/runner/
/app/assets/javascripts/runner -> /app/assets/javascripts/ci/runner
query_path = 'runner/graphql/' -> query_path = 'ci/runner/graphql/'
fixtures_path = 'graphql/runner/' -> fixtures_path = 'graphql/ci/runner/'
test_fixtures/graphql/runner -> test_fixtures/graphql/ci/runner
jest/runner -> jest/ci/runner

# After-replacement commands: 

bundle exec rspec -f d ee/spec/frontend/fixtures/runner.rb spec/frontend/fixtures/runner.rb
yarn jest --watch

# Rollback:

git reset --hard origin/master
rm -rf app/assets/javascripts/ci/
rm -rf spec/frontend/ci/

Screenshots or screen recordings

NA

How to set up and validate locally

I expanded the pipeline to run extensive tests: pipeline:run-all-jest pipeline:run-as-if-foss pipeline:run-all-rspec

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Miguel Rincon

Merge request reports

Loading