Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Memoize CI config node validator to prevent leaksfix/improve-ci-node-validatable-to-prevent-memory-leak | Grzegorz Bizon | 2016-08-09 | 1 | -6/+4 |
| | |||||
* | Merge branch 'rubocop/enable-access-modifiers-cops' into 'master' | Robert Speicher | 2016-07-29 | 1 | -1/+1 |
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enable Rubocop cops that check access modifiers ## What does this MR do? This MR enables Rubocop cops that detect methods that should be restricted but are the part of public API because of access modifiers used improperly. This also fixes existing offenses. ## Why was this MR needed? Some method in our codebase are public instead of being private because it is sometimes difficult to get it right without static analysis. ## What are the relevant issue numbers? See #17478 Closes #17372 See merge request !5014 | ||||
| * | Enable Rubocop cops for invalid access modifiersrubocop/enable-access-modifiers-cops | Grzegorz Bizon | 2016-07-19 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This enables following cops: Check for useless access modifiers Lint/UselessAccessModifier Checks for attempts to use `private` or `protected` to set the visibility of a class method, which does not work. Lint/IneffectiveAccessModifier This also disables two false possitives in concerns. | ||||
* | | Improve code, remove unused validator, improve names | Grzegorz Bizon | 2016-07-29 | 9 | -33/+19 |
| | | |||||
* | | Move job dependencies entry to the new CI config | Grzegorz Bizon | 2016-07-20 | 1 | -1/+3 |
| | | |||||
* | | Add minor readability, style improvements in CI config | Grzegorz Bizon | 2016-07-20 | 6 | -16/+18 |
| | | |||||
* | | Merge branch 'master' into refactor/ci-config-move-job-entries | Grzegorz Bizon | 2016-07-20 | 1 | -3/+3 |
|\ \ | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (321 commits) Fix the Sentry spam from CSP violations by disabling it. Limit git rev-list output count to one in forced push check Ensure Owners are included in the scope for authorized_projects Fix alignment of icons on project page Fix ci_status_helper_spec to look for new SVGs use 2.0.5, actually (2.0.4 was a bad release) upgrade rouge to 2.0.4 Fix help page paths to make sure shortcuts and the UI help page work. fixes an issue cause by a bad merge Vertically align status icon within table Add new icons for every CI status Add global style for running icon Align running icon in merge request Add new running icon; add a bunch of styles to get svg to match existing fa icons Improve code design Fix broken builds_for_ref Move when tests before to make it no conflict with manual-actions Use value of `yaml_variables` and `when` from config_processor if undefined Add CHANGELOG entry CHANGELOG item ... Conflicts: lib/ci/gitlab_ci_yaml_processor.rb spec/lib/ci/gitlab_ci_yaml_processor_spec.rb | ||||
* | | Move job environment validation to new CI config | Grzegorz Bizon | 2016-07-19 | 3 | -6/+15 |
| | | |||||
* | | Move tags and allow_failure CI entries to new config | Grzegorz Bizon | 2016-07-18 | 1 | -0/+8 |
| | | |||||
* | | Validate allowed keys only in new CI config | Grzegorz Bizon | 2016-07-18 | 1 | -0/+5 |
| | | |||||
* | | Improve valid keys validation for CI config nodes | Grzegorz Bizon | 2016-07-18 | 4 | -15/+12 |
| | | |||||
* | | Move job artifacts configuration new CI config code | Grzegorz Bizon | 2016-07-18 | 4 | -1/+71 |
| | | |||||
* | | Move job variables config entry to new CI config | Grzegorz Bizon | 2016-07-18 | 1 | -1/+5 |
| | | |||||
* | | Move except and only job nodes to new CI config | Grzegorz Bizon | 2016-07-18 | 2 | -1/+35 |
| | | |||||
* | | Move job image and services nodes to new CI config | Grzegorz Bizon | 2016-07-18 | 1 | -1/+10 |
| | | |||||
* | | Simplify abstract class for CI config entry nodes | Grzegorz Bizon | 2016-07-15 | 4 | -39/+23 |
| | | |||||
* | | Simplify CI config and remove logical validation | Grzegorz Bizon | 2016-07-15 | 4 | -29/+10 |
| | | |||||
* | | Add metadata to new CI config and expose job name | Grzegorz Bizon | 2016-07-15 | 6 | -30/+24 |
| | | |||||
* | | Remove job cache configfrom legacy yaml processor | Grzegorz Bizon | 2016-07-14 | 1 | -1/+5 |
| | | |||||
* | | Remove references to global entry in new CI config | Grzegorz Bizon | 2016-07-14 | 2 | -2/+1 |
| | | |||||
* | | Revert logical validation in CI job stage entry | Grzegorz Bizon | 2016-07-14 | 2 | -17/+0 |
| | | |||||
* | | Revert references to global node in CI job entry | Grzegorz Bizon | 2016-07-14 | 1 | -16/+4 |
| | | |||||
* | | Improve CI job entry validations in new config | Grzegorz Bizon | 2016-07-14 | 4 | -17/+31 |
| | | |||||
* | | Expose CI job commands and use in legacy processor | Grzegorz Bizon | 2016-07-13 | 1 | -7/+8 |
| | | |||||
* | | Add before script and commands to CI job entry | Grzegorz Bizon | 2016-07-13 | 1 | -2/+13 |
| | | |||||
* | | Fabricate CI entry with value, set attributes later | Grzegorz Bizon | 2016-07-13 | 1 | -8/+20 |
| | | |||||
* | | Improve CI stage configuration entry validations | Grzegorz Bizon | 2016-07-13 | 2 | -8/+10 |
| | | |||||
* | | Do not raise when getting value of invalid CI node | Grzegorz Bizon | 2016-07-12 | 1 | -1/+1 |
| | | |||||
* | | Fix using `try` on delegators in CI config entries | Grzegorz Bizon | 2016-07-12 | 1 | -2/+2 |
| | | | | | | | | | | | | See: https://github.com/rails/rails/commit/af53280a4b5b3323ac87dc60deb2b1b781197b2b | ||||
* | | Simplify undefined node definition in CI config | Grzegorz Bizon | 2016-07-12 | 5 | -61/+28 |
| | | |||||
* | | Add CI config node that is unspecified null entry | Grzegorz Bizon | 2016-07-12 | 1 | -0/+30 |
| | | |||||
* | | Prevalidate CI entries recursively on processed | Grzegorz Bizon | 2016-07-12 | 1 | -3/+2 |
| | | |||||
* | | Require parent when using node factory in CI config | Grzegorz Bizon | 2016-07-10 | 4 | -13/+24 |
| | | |||||
* | | Rename CI config job script entry node to commands | Grzegorz Bizon | 2016-07-10 | 2 | -3/+7 |
| | | |||||
* | | Move after script CI job confg to new processor | Grzegorz Bizon | 2016-07-10 | 1 | -2/+6 |
| | | |||||
* | | Add CI job script node in new config processor | Grzegorz Bizon | 2016-07-10 | 2 | -3/+38 |
| | | |||||
* | | Add before_script node to CI job entry config | Grzegorz Bizon | 2016-07-10 | 1 | -2/+16 |
| | | |||||
* | | Extend CI job entries fabrication and validation | Grzegorz Bizon | 2016-07-09 | 4 | -8/+16 |
| | | |||||
* | | Add comment for deprecated CI config `types` entry | Grzegorz Bizon | 2016-07-09 | 1 | -2/+9 |
| | | |||||
* | | Use node factory to assemble global CI config entry | Grzegorz Bizon | 2016-07-09 | 2 | -15/+23 |
| | | |||||
* | | Integrate CI job stage entry into CI configuration | Grzegorz Bizon | 2016-07-09 | 3 | -8/+38 |
| | | |||||
* | | Improve name of CI entry validation context hook | Grzegorz Bizon | 2016-07-08 | 3 | -3/+3 |
| | | |||||
* | | Add CI config known stage validation for job stage | Grzegorz Bizon | 2016-07-08 | 1 | -0/+13 |
| | | |||||
* | | Extract CI entry node validator and improve naming | Grzegorz Bizon | 2016-07-08 | 4 | -24/+25 |
| | | |||||
* | | Extract global CI config entry configuration setup | Grzegorz Bizon | 2016-07-08 | 1 | -9/+12 |
| | | |||||
* | | Extract internal attributes validator for CI entry | Grzegorz Bizon | 2016-07-08 | 2 | -7/+10 |
| | | |||||
* | | Require reference to CI config for some entries | Grzegorz Bizon | 2016-07-07 | 1 | -0/+7 |
| | | |||||
* | | Improve creating CI config entries for jobs config | Grzegorz Bizon | 2016-07-07 | 1 | -6/+10 |
| | | |||||
* | | Simplify CI config entry node factory, use attribs | Grzegorz Bizon | 2016-07-07 | 6 | -35/+30 |
| | | |||||
* | | Add scaffold of CI config for the job stage entry | Grzegorz Bizon | 2016-07-07 | 1 | -0/+22 |
| | |