Skip to content

phased-updates: first pass of technical updates #135

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 1 commit into from
Aug 6, 2025

Conversation

Hyask
Copy link
Contributor

@Hyask Hyask commented Aug 5, 2025

That documentation is written like a live document, more or less playing the role of a bug tracker with open questions and future work. This doesn't fit a documentation page, so I've removed most of those entries, and corrected what I could so that at least the technical information in that page is correct. This still needs an overhaul to better fit modern documentation standards.

cc: @s-makin

@Hyask Hyask requested review from s-makin and rkratky as code owners August 5, 2025 17:10
Copy link
Collaborator

@s-makin s-makin left a comment

Choose a reason for hiding this comment

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

Thanks for reviewing this @Hyask - I agree on dropping the block you suggested, if you want to do that as part of this PR I'll hold off on merging, otherwise I can do it as a followup (lmk which you prefer)

Copy link
Collaborator

@rkratky rkratky left a comment

Choose a reason for hiding this comment

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

Thanks, @Hyask. A few suggestions and a question.

That documentation is written like a live document, more or less playing
the role of a bug tracker with open questions and future work. This
doesn't fit a documentation page, so I've removed most of those entries,
and corrected what I could so that at least the technical information in
that page is correct. This still needs an overhaul to better fit modern
documentation standards.
@Hyask Hyask force-pushed the skia/phased_updates branch from b3753c8 to b818469 Compare August 6, 2025 13:32
@Hyask
Copy link
Contributor Author

Hyask commented Aug 6, 2025

Thanks both for the review. I've addressed your comments and suggestion, added a link to the phased-updater script, and dropped a few blocks.
I still think this would need some proper structure from a technical author, as right now, this still feel like half a spec, half a bug tracker, and another half of end-user documentation. That many halves can't be good for sure 😀

@rkratky rkratky merged commit fced34c into ubuntu:main Aug 6, 2025
1 check passed
@s-makin s-makin mentioned this pull request Aug 8, 2025
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.

3 participants