Skip to content

Dealias types when collecting parents #501

New issue

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

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

Already on GitHub? # to your account

Merged
merged 1 commit into from
Aug 25, 2021
Merged

Conversation

japgolly
Copy link
Contributor

Fixes #500

@japgolly japgolly requested a review from armanbilge August 25, 2021 02:42
Copy link
Member

@armanbilge armanbilge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh, because it was extending the type alias. Makes sense 👍

@japgolly
Copy link
Contributor Author

@armanbilge Yeah also pulls in other inherited members which was a big whoopsey.

I'll merge cos I see your approval (dude, let just merge after we assign one and other for review) and in response to your question in the other PR: yeah cool, let's get a 1.2.0 out! You wanna do it?

@japgolly japgolly merged commit 783f8c8 into series/1.x Aug 25, 2021
@japgolly japgolly deleted the topic/apiDealias branch August 25, 2021 02:54
@armanbilge
Copy link
Member

I'll merge cos I see your approval (dude, let just merge after we assign one and other for review)

Fine by me! Unless its straightforward I usually hesitate cause at least I sometimes have one more post-review afterthought to slip in 😆

yeah cool, let's get a 1.2.0 out! You wanna do it?

Yep, it's just making a tag right! Do we want to try and fix the auto-release drafter thingy first? I think it's the quickest way to make sure we acknowledge all the contributions.

@japgolly
Copy link
Contributor Author

japgolly commented Aug 25, 2021

Yep, it's just making a tag right! Do we want to try and fix the auto-release drafter thingy first? I think it's the quickest way to make sure we acknowledge all the contributions.

I couldn't be arsed doing it myself (because it's something new that wasn't present for all of the previous releases, so don't mind skipping until 2.0), but OTOH if you're free and willing to do that it would also be appreciated so, up to you :) I've got sick kids this week so I'll probably have to relieve my wife soon anyway. Can I just ask, once you've done the release, would you mind doing announcement comms? I usually just do Twitter and Reddit. Should we do more like https://users.scala-lang.org/ ? I'll leave it to you and your judgement :)

@armanbilge
Copy link
Member

Thanks, leave it to me! The scala-users post you made got a lot of views so seems a good venue there and I can also post on the Scala.js Discord/gitter, I think you have more twitter followers though :)

Hope your kids feel better soon!! Don't get sick yourself :)

@japgolly
Copy link
Contributor Author

Thanks @armanbilge !!

I think you have more twitter followers though

yeah cc me or something and I'll signal boost 👍

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

API summary lists ServiceWorkerContainer as a Scala trait
2 participants