Skip to content

macos duckplayer #459

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

Merged
merged 5 commits into from
Jun 12, 2023
Merged

macos duckplayer #459

merged 5 commits into from
Jun 12, 2023

Conversation

shakyShane
Copy link
Contributor

@shakyShane shakyShane commented Apr 22, 2023

@jonathanKingston time to get this reviewed/merged

  • added a new artifact, apple-isolated (Duck Player Overlays being the only feature in it)
  • most of the code change is ensuring this new artifact runs the same tests as windows
  • It's safe to merge here, since macOS/BrowserServicesKit will not be loading this file yet
  • The way we designed the remote config means it's safe to merge without it accidentally causing any side effects - even if they did load it
  • I removed the side-effect of registering the custom-element as we did recently with Click To Load

@shakyShane
Copy link
Contributor Author

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

Base automatically changed from shane/dom-string to main April 22, 2023 18:26
@shakyShane shakyShane force-pushed the shane/macos-duckplayer branch from b0ad2bb to d00eebd Compare April 22, 2023 21:34
Copy link
Contributor

@jonathanKingston jonathanKingston left a comment

Choose a reason for hiding this comment

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

Some minor changes needed :)

Thanks

@shakyShane
Copy link
Contributor Author

Sorry should have marked as draft, not quite ready for review - although the points raised are still valid, thanks :)

@shakyShane shakyShane marked this pull request as draft April 23, 2023 20:19
@shakyShane shakyShane force-pushed the shane/macos-duckplayer branch 3 times, most recently from bab287d to a2933e8 Compare May 3, 2023 11:59
@shakyShane shakyShane force-pushed the shane/macos-duckplayer branch 3 times, most recently from 14c4132 to 0fdd364 Compare May 7, 2023 08:50
@shakyShane shakyShane force-pushed the shane/macos-duckplayer branch 2 times, most recently from 5e651c4 to 03e0623 Compare May 28, 2023 18:08
@shakyShane shakyShane force-pushed the shane/macos-duckplayer branch 5 times, most recently from c71facd to 02c4f37 Compare June 2, 2023 14:04
@shakyShane shakyShane force-pushed the shane/macos-duckplayer branch 2 times, most recently from b3c4fdb to c642191 Compare June 8, 2023 02:34
@shakyShane shakyShane force-pushed the shane/macos-duckplayer branch from c642191 to 1acfdf2 Compare June 8, 2023 02:44
@shakyShane shakyShane marked this pull request as ready for review June 8, 2023 02:47
Copy link
Contributor

@jonathanKingston jonathanKingston left a comment

Choose a reason for hiding this comment

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

Few comments and then should be good to go

@shakyShane
Copy link
Contributor Author

@jonathanKingston - should be everything addressed now - I'll create a follow-up task in asana to review messaging construction once both mine + @franfaccin 's PRs have landed

Copy link
Contributor

@jonathanKingston jonathanKingston left a comment

Choose a reason for hiding this comment

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

LGTM

@shakyShane shakyShane merged commit 687d4ad into main Jun 12, 2023
@shakyShane shakyShane deleted the shane/macos-duckplayer branch June 12, 2023 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants