Skip to content
This repository was archived by the owner on Aug 3, 2026. It is now read-only.

fix: Use repo owner user for git operations - #190

Merged
Agrendalath merged 1 commit into
opencraft-release/lilac.2from
kshitij/fix-git-clone
Apr 14, 2022
Merged

fix: Use repo owner user for git operations#190
Agrendalath merged 1 commit into
opencraft-release/lilac.2from
kshitij/fix-git-clone

Conversation

@xitij2000

@xitij2000 xitij2000 commented Apr 13, 2022

Copy link
Copy Markdown
Member

Cherry pick of: openedx-unsupported#6717

Configuration Pull Request

Make sure that the following steps are done before merging

  • A devops team member has commented with 👍
  • are you adding any new default values that need to be overridden when this goes live?
    • Open a ticket (DEVOPS) to make sure that they have been added to secure vars.
    • Add an entry to the CHANGELOG.
  • Are you adding/updating any variables that need to be added/updated to a specific configuration-secure repo?

@xitij2000 xitij2000 changed the title fix: git clone fix: Use repo owner user for git operations Apr 13, 2022

@Agrendalath Agrendalath left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

  • I tested this: verified it's a clean cherry-pick, read the vulnerability announcement
  • I read through the code
  • I checked for accessibility issues: n/a
  • Includes documentation: n/a
  • I made sure any change in configuration variables is reflected in the corresponding client's configuration-secure repository: n/a

@Agrendalath
Agrendalath merged commit 8107593 into opencraft-release/lilac.2 Apr 14, 2022
@Agrendalath
Agrendalath deleted the kshitij/fix-git-clone branch April 14, 2022 10:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants