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

Backport minor changes from #2656 (Custom Tags) #3834

Merged
merged 6 commits into from
May 7, 2021
Merged

Backport minor changes from #2656 (Custom Tags) #3834

merged 6 commits into from
May 7, 2021

Conversation

uklotzde
Copy link
Contributor

@uklotzde uklotzde commented May 6, 2021

A first set of minor changes backported from #2656. Please refer to the individual commit messages. Some extensions are not used yet, but extracting them beforehand helps to shrink the scope of #2656.

More PRs to follow. One after another.

@uklotzde uklotzde added this to the 2.4.0 milestone May 6, 2021
@uklotzde
Copy link
Contributor Author

uklotzde commented May 6, 2021

I have deliberately decided not to base this PR on 2.3, even though it would be possible. Let's release 2.3 as is and restrict all future work to main.

@uklotzde
Copy link
Contributor Author

uklotzde commented May 6, 2021

It is annoying that all CI actions in forked repos are failing since recently:

https://github.com/uklotzde/mixxx/actions/runs/818369140

Is anyone aware of this?

@Holzhaus
Copy link
Member

Holzhaus commented May 7, 2021

It is annoying that all CI actions in forked repos are failing since recently:

https://github.com/uklotzde/mixxx/actions/runs/818369140

Is anyone aware of this?

I'll fix it. In thr meantime, just push some tags to your fork.

@daschuer
Copy link
Member

daschuer commented May 7, 2021

I had also already a look, just building in: https://github.com/daschuer/mixxx/runs/2525042562

@daschuer
Copy link
Member

daschuer commented May 7, 2021

@daschuer daschuer self-requested a review May 7, 2021 14:44
@daschuer
Copy link
Member

daschuer commented May 7, 2021

Thank you.

@daschuer daschuer merged commit 7314a96 into mixxxdj:main May 7, 2021
@uklotzde uklotzde deleted the pre-track-metadata branch May 7, 2021 15:34
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.

3 participants