[DiscordArchive] Yes <@155500896820789248> ?
[DiscordArchive] Yes <@155500896820789248> ?
Archived author: Elmsroth • Posted: 2024-02-22T16:31:47.317000+00:00
Original source
Yes <@155500896820789248> ?
Archived author: Elmsroth • Posted: 2024-02-22T16:31:53.789000+00:00
Original source
Nice to see ya here :p
Archived author: Natrist • Posted: 2024-02-22T16:31:57.900000+00:00
Original source
I believe I fixed the only missing occurence of the replaced member variable from my PR on which you commented earlier.
Archived author: Natrist • Posted: 2024-02-22T16:32:33.958000+00:00
Original source
Merging it would be a nice quality of life improvement for me, as I'm getting the same error every time I rebase my work branch on the project's master.
Archived author: Natrist • Posted: 2024-02-22T16:32:45.459000+00:00
Original source
I'll force push in a minute.
Archived author: Elmsroth • Posted: 2024-02-22T16:33:03.845000+00:00
Original source
Perhaps try to ping a maintainer of the project to quickly merge that.
Archived author: Natrist • Posted: 2024-02-22T16:33:20.809000+00:00
Original source
They can take their sweet time.
Archived author: Natrist • Posted: 2024-02-22T16:33:51.165000+00:00
Original source
However, I was going to mention that I'm working on many big updates to add compatibility with my WowGM client that I'm going to release soon™.
Archived author: Elmsroth • Posted: 2024-02-22T16:34:09.551000+00:00
Original source
:p
Do you know if there any policy regarding commit squashing in PR ?
I mean I often tries when possible to squash all commits in one to make the commit tree as clean as possible
Archived author: Natrist • Posted: 2024-02-22T16:34:28.909000+00:00
Original source
It might be worth checking out my changes at https://github.com/azerothcore/azerothco...atrist-dev