Skip to content

Update default initial branch name in group::editor code[RUN ALL RSPEC] [RUN AS-IF-FOSS]

Denys Mishunov requested to merge 326997-master-to-main-ide into master

What does this MR do?

Replaces instances of master with either a default branch name coming from the project settings or directly with main where appropriate.

The current state of the WebIDE app (/app/assets/javascripts/editor/) code [note: the existing references point to the URLs that we can not change] :

➜  gitlab git:(326997-master-to-main-ide)grep -r --include=\* master ./app/assets/javascripts/ide
./app/assets/javascripts/ide/lib/languages/hcl.js: *  Licensed under the MIT License. See https://github.com/microsoft/monaco-languages/blob/master/LICENSE.md
./app/assets/javascripts/ide/lib/languages/README.md:The Web IDE currently supports all languages defined in the [monaco-languages](https://github.com/microsoft/monaco-languages/tree/master/src) repository.
./app/assets/javascripts/ide/lib/languages/vue.js: *  Licensed under the MIT License. See https://github.com/microsoft/monaco-languages/blob/master/LICENSE.md
./app/assets/javascripts/ide/lib/languages/vue.js:// Based on handlebars template in https://github.com/microsoft/monaco-languages/blob/master/src/handlebars/handlebars.ts
./app/assets/javascripts/ide/lib/themes/monokai.js:https://github.com/brijeshb42/monaco-themes/blob/master/themes/Tomorrow-Night.json
./app/assets/javascripts/ide/lib/themes/solarized_light.js:https://github.com/brijeshb42/monaco-themes/blob/master/themes/Solarized-dark.json
./app/assets/javascripts/ide/lib/themes/white.js:    // https://github.com/Microsoft/vscode/blob/master/src/vs/editor/standalone/common/themes.ts
./app/assets/javascripts/ide/lib/themes/white.js:    // License: MIT (https://github.com/microsoft/vscode/blob/master/LICENSE.txt)
./app/assets/javascripts/ide/lib/themes/white.js:    // GitHub theme based on https://github.com/brijeshb42/monaco-themes/blob/master/themes/GitHub.json
./app/assets/javascripts/ide/lib/themes/white.js:    // License: MIT (https://github.com/brijeshb42/monaco-themes/blob/master/LICENSE)
./app/assets/javascripts/ide/lib/themes/solarized_dark.js:https://github.com/brijeshb42/monaco-themes/blob/master/themes/Solarized-dark.json
./app/assets/javascripts/ide/lib/themes/dark.js:https://github.com/brijeshb42/monaco-themes/blob/master/themes/Tomorrow-Night.json

The tests' (/spec/frontend/ide) state:

➜  gitlab git:(326997-master-to-main-ide)grep -r --include=\* master ./spec/frontend/ide
➜  gitlab git:(326997-master-to-main-ide)

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #326997 (closed)

Edited by Jose Ivan Vargas

Merge request reports