Skip to content

Conversation

silverwind
Copy link
Member

Synced the list to what is allowed on GitHub currently.

@silverwind silverwind changed the title Update default allowed attchment types to GitHub Update default allowed attchment types Jul 1, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jul 1, 2022
@silverwind silverwind changed the title Update default allowed attchment types Update default allowed attachment types Jul 1, 2022
Synced the list to what is allowed on GitHub currently.
@lunny lunny added the pr/breaking Merging this PR means builds will break. Needs a description what exactly breaks, and how to fix it! label Jul 1, 2022
@lunny lunny added this to the 1.18.0 milestone Jul 1, 2022
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jul 1, 2022
@silverwind
Copy link
Member Author

silverwind commented Jul 1, 2022

Breaking label? The new list a a complete superset of the old one (.log was sorted correctly).

@lunny lunny removed the pr/breaking Merging this PR means builds will break. Needs a description what exactly breaks, and how to fix it! label Jul 1, 2022
@6543
Copy link
Member

6543 commented Jul 1, 2022

🚀

@6543 6543 merged commit 54e7483 into go-gitea:main Jul 1, 2022
@silverwind silverwind deleted the allowed-types branch July 1, 2022 14:18
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jul 1, 2022
* upstream/main:
  Update default allowed attachment types (go-gitea#20192)
vsysoev pushed a commit to IntegraSDL/gitea that referenced this pull request Aug 10, 2022
Synced the list to what is allowed on GitHub currently.
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants