summaryrefslogtreecommitdiff
path: root/doc/development
Commit message (Collapse)AuthorAgeFilesLines
* Corrected links/usernames in performance guideupdate-performance-guideYorick Peterse2016-08-111-5/+4
| | | | | | | This fixes two broken links in the performance guide and removes the mention of Josh as he no longer works for GitLab. [ci skip]
* Merge branch 'add-column-with-default-to-downtime-guide' into 'master' Achilleas Pipinellis2016-08-101-0/+8
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Mention add_column_with_default in downtime guide ## What does this MR do? This extends the "What causes downtime" guide with a mention of the `add_column_with_default` migration helper. ## Are there points in the code the reviewer needs to double check? Spelling, grammar, etc. ## Why was this MR needed? Currently the guide may lead one to believe it's not possible at all to add a column with a default value. ## Does this MR meet the acceptance criteria? - [x] ~~[CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added~~ - [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - Tests - [ ] All builds are passing - [ ] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [ ] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !5719
| * Mention add_column_with_default in downtime guideadd-column-with-default-to-downtime-guideYorick Peterse2016-08-101-0/+8
| | | | | | | | [ci skip]
* | Merge branch 'update-ui-guide--svgs' into 'master' Dmitriy Zaporozhets2016-08-101-0/+36
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update UI Guide with SVG guidelines This addition to the guide is to provide some guidelines to UX designers when exporting SVGs. Please let me know if anything is unclear or if you any improvements so we can document it clearly for everyone. cc / @hazelyang @cperessini @dimitrieh @connorshea @annabeldunstone @dzaporozhets ## What are the relevant issue numbers? https://gitlab.com/gitlab-org/gitlab-ee/issues/872 See merge request !5748
| * | spellcheckupdate-ui-guide--svgsTaurie Davis2016-08-091-2/+2
| | |
| * | Add svg guidelines to ui guideTaurie Davis2016-08-091-0/+36
| | |
* | | Small refactor of doc/development/README.mdAchilleas Pipinellis2016-08-101-4/+3
| | | | | | | | | | | | | | | | | | - Fix wrong doc_styleguide URL - Clarify migrations refer to SQL - Capitalize CSS
* | | Merge branch 'link-to-gdk-howtos' into 'master' Achilleas Pipinellis2016-08-101-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Link to doc root, not readme of GDK I hope this makes navigation between gitlab-ce developer docs and GDK docs more smooth. See merge request !5716
| * | | Link to doc root, not readme of GDKJacob Vosmaer2016-08-081-1/+1
| | |/ | |/|
* | | Merge branch 'api-examples-curl-long-options' into 'master' Achilleas Pipinellis2016-08-101-7/+7
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use long options for curl examples in documentation ## What does this MR do? Use long options (e.g. `--header` instead of `-H`) for curl examples in documentation. ## Why was this MR needed? Short options are less readable. ## What are the relevant issue numbers? https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5465#note_13603730 See merge request !5703
| * | use long options for curl examples in API documentation (!5703)winniehell2016-08-091-7/+7
| |/
* | Merge branch 'dz-newlines-styleguide' into 'master' Robert Speicher2016-08-081-0/+102
|\ \ | |/ |/| | | | | Add newlines styleguide for Ruby code See merge request !5636
| * Add newlines styleguide for Ruby codedz-newlines-styleguideDmitriy Zaporozhets2016-08-051-0/+102
| |
* | Simplify feature introduction notesimplify_intro_features_docAchilleas Pipinellis2016-08-081-5/+5
| | | | | | | | [ci skip]
* | Merge branch 'doc-location-guidelines' into 'master' Achilleas Pipinellis2016-08-061-5/+60
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add doc guidelines on documents naming and location ## What does this MR do? Add guidelines on the structure of the documentation. ## What are the relevant issue numbers? https://gitlab.com/gitlab-org/gitlab-ce/issues/3349 See merge request !5641
| * | Add doc guidelines on documents naming and locationdoc-location-guidelinesAchilleas Pipinellis2016-08-031-5/+60
| | | | | | | | | | | | [ci skip]
* | | Add 'run tests' docs from GDKJacob Vosmaer2016-08-051-4/+26
| | |
* | | Added guide about migrations and downtimedatabase-guidesYorick Peterse2016-08-042-0/+154
| | | | | | | | | | | | | | | This guide describes various commonly used operations and the impact they have on the availability of a GitLab instance.
* | | Merge branch 'coffescript_docs_migration' into 'master' Rémy Coutable2016-08-041-2/+3
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update docs for CoffeeScript -> JavaScript. ## What does this MR do? Removes some CoffeeScript docs and adds more JS ones ## Are there points in the code the reviewer needs to double check? No ## Why was this MR needed? To migrate some docs ## What are the relevant issue numbers? #20098 ## Screenshots (if relevant) None ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [ ] API support added - Tests - [ ] Added for this feature/bug - [ ] All builds are passing - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !5433
| * | | Remove CHANGELOG and fix typoRémy Coutable2016-08-041-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | | Update docs for CoffeeScript -> JavaScript.Jacob Schatz2016-07-221-2/+3
| | |/ | |/|
* | | Structure the development documentationSid Sijbrandij2016-08-041-7/+26
| |/ |/|
* | API methods should be documented using Grape's DSL document-grape-dslRémy Coutable2016-08-011-0/+6
|/ | | See https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/2397#note_13491048
* Merge branch 'email-domain-blacklist' into 'master' Robert Speicher2016-07-201-1/+1
|\ | | | | | | | | | | | | | | | | | | | | Added the ability to block sign ups using a domain blacklist. As part of this MR, I restructured the Application Settings form to separate **Sign up** related settings from **Sign in** related settings and make everything cleaner and easier to read. Fixes #19749 Related to #5573 See merge request !5259
| * Refactor and rename `restricted_signup_domains` to `domain_whitelist` to ↵Patricio Cano2016-07-181-1/+1
| | | | | | | | better conform to its behavior and newly introduced behavior.
* | Added checks for migration downtimemigration-downtime-tagsYorick Peterse2016-07-201-23/+23
| | | | | | | | | | | | | | | | | | | | | | These new checks can be used to check if migrations require downtime or not (as tagged by their authors). In CI this compares the current branch with master so migrations added by merge requests are automatically verified. To check the migrations added since a Git reference simply run: bundle exec rake gitlab:db:downtime_check[GIT_REF]
* | Upgrade Gemojione from 2.6.1 to 3.0.1.Connor Shea2016-07-181-0/+5
|/ | | | | | | | | | | | | | | | This adds the 2016 emoji as well as support for using SVG images instead of PNGs. It also fixes a number of incorrectly categorized emoji and other minor issues. Upgrade Rake task for Gemojione 3.0.0 and generate sprites. Upgrade aliases.json by pulling down index.json from the gemojione repository and running the generate_aliases.rb file. Changelog: https://github.com/jonathanwiesel/gemojione/blob/master/CHANGELOG.md#v301-2016-07-16 For the specific emoji added to the Unicode standard, see: http://emojione.com/releases/2.2.4/ Huge kudos to Jonathan Wiesel (@jonathanwiesel) for his work on the gemojione gem!
* Don't ask Heather to review documentation MR'sdont-ask-nearlythereJacob Vosmaer (GitLab)2016-07-151-1/+1
|
* Add a UI guide note about number_with_delimiterrs-delimit-in-ui-guideRobert Speicher2016-07-121-21/+27
| | | | [ci skip]
* Improve wording in UI guideDmitriy Zaporozhets2016-07-121-1/+1
|
* Fix typo in UI guidedz-ui-guide-navDmitriy Zaporozhets2016-07-111-1/+1
|
* Add rule about adding new header tab to the ui guideDmitriy Zaporozhets2016-07-111-0/+2
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Update ui_guide.md with button capitalize ruledz-button-capitalizeDmitriy Zaporozhets2016-07-081-1/+2
|
* Merge branch 'fix-dead-links' into 'master' Achilleas Pipinellis2016-07-041-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix dead links in the docs (Jun 2016) ## What does this MR do? Fix dead links in the docs found as of Jun 2016. ## Are there points in the code the reviewer needs to double check? n/a ## Why was this MR needed? This MR must improve UX on docs.gitlab.com. ## What are the relevant issue numbers? Closes #19156 ## Does this MR meet the acceptance criteria? - No [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) updated - [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !4921
| * Fix dead links in the docsTakuya Noguchi2016-06-261-1/+1
| |
* | optimize png images losslessly using zopflipngPeter Dave Hello2016-06-291-0/+0
| |
* | Merge branch 'gitlab_architecture' into 'master' Achilleas Pipinellis2016-06-283-1/+3
|\ \ | |/ |/| | | | | | | | | Refactor GitLab architecture document Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/14236 See merge request !3694
| * Add "GitLab team members only" to diagram linkgitlab_architectureChris Wilson2016-06-281-1/+1
| |
| * Update GitLab architecture diagram, include Google draw linkChris Wilson2016-06-232-0/+2
| | | | | | | | [ci skip]
| * Replace GitLab architecture diagramAchilleas Pipinellis2016-04-133-1/+1
| | | | | | | | [ci skip]
* | Merge branch 'hamlit-test' into 'master' Robert Speicher2016-06-231-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace Haml with Hamlit What this change does: - It replaces HAML with [Hamlit](https://github.com/k0kubun/hamlit), a much more efficient implementation of HAML. - It removes `haml-rails`. - It adds `hamlit.rb` and removes `haml.rb`. - It makes things faster and less memory-hungry! See also #13201. See merge request !3666
| * | Restrict hamlit version, remove coffeescript filters.Connor Shea2016-06-231-1/+1
| | |
* | | Merge branch 'document-emoji-rake-tasks' into 'master' Achilleas Pipinellis2016-06-231-0/+20
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added basic docs for the Emoji Rake tasks ## What does this MR do? This MR adds basic documentation for the two Emoji Rake tasks. ## Are there points in the code the reviewer needs to double check? Spelling, that sort of thing. ## Why was this MR needed? These Rake tasks only had some small amount of source level documentation. ## What are the relevant issue numbers? https://gitlab.com/gitlab-org/gitlab-ce/issues/14794 ## Does this MR meet the acceptance criteria? - [x] ~~[CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added~~ - [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [ ] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !4877
| * | | Added basic docs for the Emoji Rake tasksdocument-emoji-rake-tasksYorick Peterse2016-06-231-0/+20
| |/ / | | | | | | | | | | | | | | | Fixes gitlab-org/gitlab-ce#14794 gitlab-com/performance#13 [ci skip]
* | | Remove Rubocop skip comment from migration style guide [ci skip]Drew Blessing2016-06-231-1/+0
|/ /
* | Add styleguide on configuration settings documentationAchilleas Pipinellis2016-06-201-0/+56
| | | | | | | | [ci skip]
* | Track method call times/counts as a single metricYorick Peterse2016-06-171-17/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously we'd create a separate Metric instance for every method call that would exceed the method call threshold. This is problematic because it doesn't provide us with information to accurately get the _total_ execution time of a particular method. For example, if the method "Foo#bar" was called 4 times with a runtime of ~10 milliseconds we'd end up with 4 different Metric instances. If we were to then get the average/95th percentile/etc of the timings this would be roughly 10 milliseconds. However, the _actual_ total time spent in this method would be around 40 milliseconds. To solve this problem we now create a single Metric instance per method. This Metric instance contains the _total_ real/CPU time and the call count for every instrumented method.
* | Update migration_style_guide.md with new detailsfix_migration_style_guideDrew Blessing2016-06-151-2/+14
| |
* | Instrument private/protected methodsPaco Guzman2016-06-141-2/+2
| | | | | | | | | | | | | | By default instrumentation will instrument public, protected and private methods, because usually heavy work is done on private method or at least that’s what facts is showing
* | Measure CPU time for instrumented methodsPaco Guzman2016-06-141-5/+6
| |