Skip to content

Conversation

@mattkjames7
Copy link
Contributor

Description

Minor change to set the user shell to /bin/bash for the memgraph user in line with previous releases.

@mattkjames7
Copy link
Contributor Author

mattkjames7 commented Mar 26, 2025

Pull request type

  • Bugfix
  • Algorithm/Module
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Related issues

Delete if this PR doesn't resolve any issues. Link the issue if it does.

######################################

Reviewer checklist (the reviewer checks this part)

Module/Algorithm

  • Core algorithm/module implementation
  • Query module implementation
  • Tests provided (unit / e2e)
  • Code documentation
  • README short description

Documentation checklist

  • Add the documentation label tag
  • Add the bug / feature label tag
  • Add the milestone for which this feature is intended
    • If not known, set for a later milestone
  • Write a release note, including added/changed clauses
    • Set default shell for memgraph user to /bin/bash for consistency with all previous Docker images. This should not change anything from the user's perspective #577.
  • Link the documentation PR here
    • [Documentation PR link]
  • Tag someone from docs team in the comments

@mattkjames7 mattkjames7 added the Docs unnecessary Docs unnecessary label Mar 26, 2025
@mattkjames7 mattkjames7 added this to the mage-v3.1.1 milestone Mar 26, 2025
@mattkjames7 mattkjames7 marked this pull request as ready for review March 26, 2025 12:52
@mattkjames7 mattkjames7 enabled auto-merge March 26, 2025 13:12
@sonarqubecloud
Copy link

@mattkjames7 mattkjames7 added this pull request to the merge queue Mar 26, 2025
Merged via the queue into main with commit 81e21e3 Mar 26, 2025
10 checks passed
@mattkjames7 mattkjames7 deleted the use-bash-shell branch March 26, 2025 21:55
@gitbuda gitbuda added Docs - changelog only Docs - changelog only and removed Docs unnecessary Docs unnecessary labels Mar 28, 2025
@gitbuda gitbuda mentioned this pull request Mar 28, 2025
18 tasks
@mattkjames7 mattkjames7 added the bug bug label Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug bug Docs - changelog only Docs - changelog only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants