Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

fix: update CODEOWNERS #10

Merged
merged 1 commit into from
Mar 24, 2023
Merged

Conversation

sebastiendan
Copy link
Member

@sebastiendan sebastiendan commented Mar 24, 2023

Description

CODEOWNERS attributes docker/local to Tools & Protocol teams, but we're having our updates in docker/local-topos. This PR replaces the former with the broader docker path.

Additions and Changes

  • Update CODEOWNERS to replace docker/local with docker

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

to include the whole docker folder
@sebastiendan sebastiendan requested a review from a team as a code owner March 24, 2023 10:52
@sebastiendan sebastiendan self-assigned this Mar 24, 2023
@sebastiendan sebastiendan merged commit e716359 into develop Mar 24, 2023
@sebastiendan sebastiendan deleted the fix/update-codeowners-docker branch March 24, 2023 20:41
atanmarko pushed a commit that referenced this pull request Aug 18, 2023
atanmarko pushed a commit that referenced this pull request Sep 7, 2023
atanmarko pushed a commit that referenced this pull request Sep 15, 2023
atanmarko pushed a commit that referenced this pull request Oct 9, 2023
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