Branch: refs/heads/master
Home: https://github.com/openSUSE/open-build-service
Commit: efd48d5f3d5a231b84a068a763d943bf1b9f4e6f
https://github.com/openSUSE/open-build-service/commit/efd48d5f3d5a231b84a068...
Author: Dany Marcoux
Date: 2020-06-29 (Mon, 29 Jun 2020)
Changed paths:
M .gitignore
M dist/README.UPDATERS
M dist/obs-server.spec
M src/api/.rubocop.yml
M src/api/app/models/user.rb
M src/api/config/application.rb
A src/api/db/schema.rb
R src/api/db/structure.sql
Log Message:
-----------
Replace structure.sql by schema.rb
We don't use any database-specific features. This will simplify
maintenance since we don't have to deal with all the tiny details of a
SQL dump (like changes between various MariaDB versions)
Commit: 07968b95e6cd9e2a6eedad14648c1ff5c54128af
https://github.com/openSUSE/open-build-service/commit/07968b95e6cd9e2a6eedad...
Author: Dany Marcoux
Date: 2020-06-29 (Mon, 29 Jun 2020)
Changed paths:
M .circleci/config.yml
M src/api/.rubocop_todo.yml
M src/api/lib/tasks/databases.rake
M src/api/lib/tasks/dev.rake
M src/api/script/api_minitest.sh
R src/api/script/compare_structure_sql.sh
R src/api/test/dump_2.5.sql.xz
Log Message:
-----------
Remove custom database rake tasks
`db:structure:dump` is a monkey patch and we don't need it anymore since
we're using schema.rb.
`db:structure:verify` is testing if the migrations were executed. It is useful,
but it can be replaced by `db:abort_if_pending_migrations`.
`db:structure:verify_no_bigint` isn't needed since primary keys in migrations are
explicitely set to use `:integer` by default. See
https://github.com/openSUSE/open-build-service/blob/63caf6b6cc601acf818a22c9...
`db:migrate` is somewhat useful, reminding us about running data
migrations. We keep it.
Commit: 1fe9a76f50b91d1caec606b37a25327f23ec154a
https://github.com/openSUSE/open-build-service/commit/1fe9a76f50b91d1caec606...
Author: Dany Marcoux
Date: 2020-06-29 (Mon, 29 Jun 2020)
Changed paths:
M .rubocop_todo.yml
M src/api/.rubocop_todo.yml
Log Message:
-----------
Regenerate RuboCop TODOs
With the introduction of schema.rb, some RuboCop cops are now active and
linting the schema for potential offenses. One of them
(Rails/UniqueValidationWithoutIndex) is registering offenses and to keep
changes as small as possible, we don't want to address the offenses in
this PR. We could have disabled the cops, but they do provide value. In
the end, regenerating the RuboCop TODOs makes the CI pass and allows us
to tackle those offenses in a separate PR.
Commit: ae7aa042637a2ae9382364d0b1ca42627d8fc8f4
https://github.com/openSUSE/open-build-service/commit/ae7aa042637a2ae9382364...
Author: Henne Vogelsang
Date: 2020-07-01 (Wed, 01 Jul 2020)
Changed paths:
M .circleci/config.yml
M .gitignore
M .rubocop_todo.yml
M dist/README.UPDATERS
M dist/obs-server.spec
M src/api/.rubocop.yml
M src/api/.rubocop_todo.yml
M src/api/app/models/user.rb
M src/api/config/application.rb
A src/api/db/schema.rb
R src/api/db/structure.sql
M src/api/lib/tasks/databases.rake
M src/api/lib/tasks/dev.rake
M src/api/script/api_minitest.sh
R src/api/script/compare_structure_sql.sh
R src/api/test/dump_2.5.sql.xz
Log Message:
-----------
Merge pull request #9852 from dmarcoux/migrate-from-structure-sql-to-schema-rb
Migrate from structure.sql to schema.rb
Compare: https://github.com/openSUSE/open-build-service/compare/ccf5032c3b6f...ae7aa0...