Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge upstream #159

Merged
merged 93 commits into from
Dec 2, 2022
Merged

Merge upstream #159

merged 93 commits into from
Dec 2, 2022

Conversation

TroyKomodo
Copy link

merge upstream

Nerixyz and others added 30 commits October 22, 2022 11:42
Co-authored-by: Rasmus Karlsson <rasmus.karlsson@pajlada.com>
Co-authored-by: Daniel Sage <24928223+dnsge@users.noreply.github.com>
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
Bumps [ZedThree/clang-tidy-review](https://github.com/ZedThree/clang-tidy-review) from 0.10.0 to 0.10.1.
- [Release notes](https://github.com/ZedThree/clang-tidy-review/releases)
- [Changelog](https://github.com/ZedThree/clang-tidy-review/blob/master/CHANGELOG.md)
- [Commits](ZedThree/clang-tidy-review@v0.10.0...v0.10.1)

---
updated-dependencies:
- dependency-name: ZedThree/clang-tidy-review
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Rasmus Karlsson <rasmus.karlsson@pajlada.com>
Co-authored-by: Rasmus Karlsson <rasmus.karlsson@pajlada.com>
Co-authored-by: 23rd <23rd@vivaldi.net>
Co-authored-by: David Myers <dave-myers@outlook.com>
Co-authored-by: Rasmus Karlsson <rasmus.karlsson@pajlada.com>
Co-authored-by: Rasmus Karlsson <rasmus.karlsson@pajlada.com>
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
Co-authored-by: Felanbird <41973452+Felanbird@users.noreply.github.com>
Co-authored-by: Rasmus Karlsson <rasmus.karlsson@pajlada.com>
fourtf and others added 15 commits November 20, 2022 20:51
…#4174)

Co-authored-by: Rasmus Karlsson <rasmus.karlsson@pajlada.com>
Co-authored-by: Felanbird <41973452+Felanbird@users.noreply.github.com>
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
Fixes Chatterino#3844
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
Co-authored-by: Felanbird <41973452+Felanbird@users.noreply.github.com>
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
fix Chatterino#3863
fix Chatterino#4180
…atterino#4187)

Co-authored-by: Felanbird <41973452+Felanbird@users.noreply.github.com>
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
Co-authored-by: Rasmus Karlsson <rasmus.karlsson@pajlada.com>
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
fixes Chatterino#2205
This change enforces strict include grouping using IncludeCategories

In addition to adding this to the .clang-format file and applying it in the tests/src and src directories, I also did the following small changes:

    In ChatterSet.hpp, I changed lrucache to a <>include
    In Irc2.hpp, I change common/SignalVector.hpp to a "project-include"
    In AttachedWindow.cpp, NativeMessaging.cpp, WindowsHelper.hpp, BaseWindow.cpp, and StreamerMode.cpp, I disabled clang-format for the windows-includes
    In WindowDescriptors.hpp, I added the missing vector include. It was previously not needed because the include was handled by another file that was previously included first.
    clang-format minimum version has been bumped, so Ubuntu version used in the check-formatting job has been bumped to 22.04 (which is the latest LTS)
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
When you start Chatterino, you get two warnings from libpng: libpng warning: iCCP: known incorrect sRGB profile.
This is because two resources shipped by Chatterino have this incorrect profile. This PR fixes that (essentially just convert file.png file.png).
* Update make-release.md documentation

* Update changelog

* Update `com.chatterino.chatterino.appdata.xml` version

* Update `Version.hpp`

* Lint make-release.md

* Lint again
@TroyKomodo TroyKomodo force-pushed the troy/merge-upstream branch 2 times, most recently from de12874 to e050547 Compare December 2, 2022 17:51
@Felanbird
Copy link

#include <optional> needs to be above #include <utility> instead of below #include "Paint.hpp" that'll make the formatter happy
image

@TroyKomodo TroyKomodo force-pushed the troy/merge-upstream branch 2 times, most recently from f6b4963 to 0a39cac Compare December 2, 2022 20:18
src/Application.cpp Outdated Show resolved Hide resolved
@TroyKomodo TroyKomodo merged commit 79c64e9 into chatterino7 Dec 2, 2022
@TroyKomodo TroyKomodo deleted the troy/merge-upstream branch December 2, 2022 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.