Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B bull
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 175
    • Issues 175
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 9
    • Merge requests 9
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • OptimalBits
  • bull
  • Merge requests
  • !1926

Clarify that the limiter max is the number of jobs, not their 1/1000ths.

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/ryanprince/develop into develop Dec 04, 2020
  • Overview 2
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: ryanprince

The thousands separator confused me for a little while --- in the United States, a . usually indicates a decimal separator. This changes the explanation so that its references to the values are identical to the values in the code example.

This should be the correct interpretation of what's written, since it's the interpretation that the test 'should obey the rate limit' in test_rate_limiter.js verifies.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/ryanprince/develop