Skip to content

WIP: api settings cleanup (EE related)

Marcel Amirault requested to merge (removed):docs-app-settings-api into master

See more details in: https://gitlab.com/gitlab-org/gitlab-ce/issues/49155

This MR will be used to collect the fixes to the various api settings files and docs (EE ONLY) that I find. It may be recreated at some point before final merge, and may need review by several people due to the wide variety of issues I've found.

File: app/helpers/application_settings_helper.rb

Order settings alphabetically (CE too)

File: lib/api/settings.rb

Fix typos:
:sidekiq_throttling_queus(CE too)
:performance_bar_allowed_group_id, type: String, desc: 'Depreated: (CE too)

File: doc/api/settings.md

Large Cleanup of table spacing, to make .md easier to read, and to make future updates simpler.(CE too)
Order settings alphabetically (CE too)
add missing help_page_text setting (CE too)
shared_runners_textand terminal_max_session_time were hidden (table formatting broken) by accident in https://gitlab.com/gitlab-org/gitlab-ee/commit/cdca89df96851cbf93c5203a96a4d3d5af316690

Edited by Marcel Amirault

Merge request reports