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

Add some initial notes for updating game-activity glue #89

Closed
wants to merge 1 commit into from

Conversation

rib
Copy link
Collaborator

@rib rib commented Jun 25, 2023

This digs up a few notes from when I first implemented that agkd-rust crate that later became android-activity.

Hopefully we can prune the set of patches that we're tracking the next time that we sync with a new AGDK GameActivity release.

TODO: It would be good to have improved workflow documentation for how we will pull in new AGDK releases.

TODO: We should also document some kind of policy for what kinds of changes will be considered semver breaking changes for the android-activity crate.

This digs up a few notes from when I first implemented that agkd-rust
crate that later became android-activity.

Hopefully we can prune the set of patches that we're tracking the
next time that we sync with a new AGDK GameActivity release.

TODO: It would be good to have improved workflow documentation for how
we will pull in new AGDK releases.

TODO: We should also document some kind of policy for what kinds of
changes will be considered semver breaking changes for the
android-activity crate.
@rib rib mentioned this pull request Jun 25, 2023
@rib
Copy link
Collaborator Author

rib commented Jul 30, 2023

Closing this for now since the merge of #88 leaves a reasonable paper trail of the steps and patches involved in updating to the latest GameActivity release. All the patches are back-to-back currently so it's quite clear what changes we're maintaining on top of the upstream GameActivity glue code.

@rib rib closed this Jul 30, 2023
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.

1 participant