Skip to content
Snippets Groups Projects

Update NPM docs to remove case restriction

Merged Steve Abrams requested to merge sabrams-master-patch-36478 into master
All threads resolved!

What does this MR do?

The changes in !53266 (merged) removed the case restriction on NPM scopes. A user owning a namespace owns it case-insensitively, meaning if I own gitlab.com/group, I also own gitlab.com/group and gitlab.com/GrOuP.

This MR removes the notes about case restrictions in the NPM scope naming convention from the docs.

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • 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
Edited by Steve Abrams

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Steve Abrams resolved all threads

    resolved all threads

  • Steve Abrams added 1 commit

    added 1 commit

    • 43cf7015 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Steve Abrams marked the checklist item Documentation (if required) as completed

    marked the checklist item Documentation (if required) as completed

  • Nick Gaskill
  • Nick Gaskill changed milestone to %13.10

    changed milestone to %13.10

  • Tim Rizzi added 1 commit

    added 1 commit

    • 3363629f - Add specifics about relaxing the naming convention

    Compare with previous version

  • Tim Rizzi added 1 commit

    added 1 commit

    • 2cb775ae - Minor updates and make npm lowercase

    Compare with previous version

  • Tim Rizzi
  • Tim Rizzi added 1 commit

    added 1 commit

    • 4a81617a - Update copy to be root of namespace

    Compare with previous version

  • Nick Gaskill
  • Nick Gaskill added 1 commit

    added 1 commit

    • d9cce6b0 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Steve Abrams assigned to @ngaskill and unassigned @sabrams

    assigned to @ngaskill and unassigned @sabrams

  • Tim Rizzi
  • Nick Gaskill approved this merge request

    approved this merge request

  • Tim Rizzi resolved all threads

    resolved all threads

  • merged

  • Nick Gaskill mentioned in commit aedb1cff

    mentioned in commit aedb1cff

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • Please register or sign in to reply
    Loading