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

Random fixes broken out of base layer work #1374

Merged

Conversation

TBBle
Copy link
Contributor

@TBBle TBBle commented Apr 26, 2022

A small batch of fixes that I've broken out of #901 to reduce the scope of that PR to only those parts that introduce not-yet-acceptible features.

This reduces the depth of the branches I have to manage and rebase.

Mostly documentation and CI, but also introduces a new SafeOpen API used by #901.

@TBBle TBBle requested a review from a team as a code owner April 26, 2022 09:37
Copy link
Contributor

@dcantah dcantah left a comment

Choose a reason for hiding this comment

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

LGTM, one bit of feedback on the safefile commit.

Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
@TBBle TBBle force-pushed the random-fixes-broken-out-of-base-layer-work branch 2 times, most recently from 716a82d to f68596e Compare April 27, 2022 13:14
Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
I also sorted the archive step list of binaries to match the calls to
`go build` so that it's clear when a binary has been overlooked.

Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
@TBBle TBBle force-pushed the random-fixes-broken-out-of-base-layer-work branch from f68596e to be9ebd6 Compare April 27, 2022 13:22
@dcantah dcantah merged commit fc59390 into microsoft:master Apr 27, 2022
@TBBle TBBle deleted the random-fixes-broken-out-of-base-layer-work branch April 28, 2022 13:15
anmaxvl added a commit that referenced this pull request Feb 7, 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.

3 participants