Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Update workers.md to make media_repository work (again) - #6519

Merged
richvdh merged 4 commits into
matrix-org:masterfrom
mnowiasz:master
Dec 11, 2019
Merged

Update workers.md to make media_repository work (again)#6519
richvdh merged 4 commits into
matrix-org:masterfrom
mnowiasz:master

Conversation

@mnowiasz

@mnowiasz mnowiasz commented Dec 11, 2019

Copy link
Copy Markdown
Contributor

Pull Request Checklist

  • Pull request is based on the develop branch
  • Pull request includes a changelog file
  • Pull request includes a sign off
  • Code style is correct (run the linters)

richvdh
richvdh approved these changes Dec 11, 2019
View reviewed changes

@richvdh richvdh 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.

thanks! Have tweaked the wording a little, otherwise looks good.

@richvdh
richvdh merged commit 58fdcbd into matrix-org:master Dec 11, 2019
@richvdh

richvdh commented Dec 11, 2019

Copy link
Copy Markdown
Member

(btw, if this wasn't just a documentation change, we'd insist on you following the guidelines in CONTRIBUTING.rst about targetting the develop branch, including a changelog fragment, and signing off your changes.)

@richvdh richvdh mentioned this pull request Dec 11, 2019
Closed
babolivier pushed a commit that referenced this pull request Sep 1, 2021
@anoadragon453
Update workers.md to make media_repository work (again) (#6519)
e71d3cd 
* commit '58fdcbdfe':
  Update  workers.md to make media_repository work (again) (#6519)
  Remove #6369 changelog
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
@richvdh richvdh richvdh approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mnowiasz @richvdh