Skip to content
Snippets Groups Projects

STRLEN

Merged STRLEN
danramteke/RediStack:strlen into master
All threads resolved!
Merged Daniel Ramteke requested to merge danramteke/RediStack:strlen into master
All threads resolved!

Motivation

STRLEN command was missing, but HSTRLEN was present.

Modifications

Added command and helper method.

Result

STRLEN command is easy to use.

Checklist

  • Copyright headers in new files (see CONTRIBUTING.md)
  • Unit tests added
  • API Documentation comments

Merge request reports

Pipeline #476007281 passed

Stage: Platform Tests
Stage: Quality Checks
Pipeline: RediStack

#476007295

    Pipeline: RediStack

    #476007301

      Pipeline: RediStack

      #476007321

        +3

        Pipeline passed for 59b00517 on danramteke:strlen

        Approved by
        Nathan Harris

        Merged by Nathan HarrisNathan Harris 3 years ago (Feb 21, 2022 7:10pm UTC)

        Merge details

        • Changes merged into master with b449334c (commits were squashed).
        • Deleted the source branch.

        Pipeline #491257804 passed

        Stage: Platform Tests
        Pipeline: RediStack

        #491257821

          Pipeline: RediStack

          #491257827

            Pipeline: RediStack

            #491257824

              +3

              Pipeline passed for b449334c on master

              Activity

              Filter activity
              • Approvals
              • Assignees & reviewers
              • Comments (from bots)
              • Comments (from users)
              • Commits & branches
              • Edits
              • Labels
              • Lock status
              • Mentions
              • Merge request status
              • Tracking
            • Nathan Harris
            • Daniel Ramteke added 1 commit

              added 1 commit

              Compare with previous version

            • Daniel Ramteke added 1 commit

              added 1 commit

              Compare with previous version

            • Nathan Harris resolved all threads

              resolved all threads

            • Nathan Harris approved this merge request

              approved this merge request

            • Nathan Harris merged

              merged

            • Nathan Harris
              Nathan Harris @Mordil
              Owner

              @danramteke Thanks for the contribution!

              If you want to backport this to 1.x to get access to it "immediately", you can create a branch off of support/1.x and create a merge request against that branch.

              • Please register or sign in to reply