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

Tidy up #187

Merged
merged 3 commits into from
Mar 8, 2024
Merged

Tidy up #187

merged 3 commits into from
Mar 8, 2024

Conversation

stagnation
Copy link
Contributor

@stagnation stagnation commented Feb 7, 2024

Some small changes to I made during the work to update windows workers.
Keep them as a separate PR to merge them sooner, and reduce the scope of the meaty review of the PR.

@stagnation stagnation changed the title Feature/tidy up 2024 01 Tidy up Feb 7, 2024
Copy link
Member

@EdSchouten EdSchouten left a comment

Choose a reason for hiding this comment

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

Thanks!

pkg/filesystem/local_directory_windows.go Outdated Show resolved Hide resolved
@stagnation stagnation force-pushed the feature/tidy-up-2024-01 branch 2 times, most recently from c2ac931 to 0beee53 Compare February 7, 2024 16:19
@stagnation
Copy link
Contributor Author

Thanks for the comments. Will address them soon. I accidentally pushed section 2 here, which I'd rather review as a stacked pull request (because it needs more review and redesign, thanks for that!). Sorry for the inconvenience here.

pkg/filesystem/local_directory_windows.go Outdated Show resolved Hide resolved
pkg/filesystem/local_directory_windows.go Outdated Show resolved Hide resolved
pkg/filesystem/local_directory_windows.go Outdated Show resolved Hide resolved
Nils Wireklint added 3 commits March 8, 2024 15:45
To work with users who prefer a different output in their personal rc
file. Using '--nohome_rc' is not good either, if they want to change the
output base this query would duplicate a lot of data.
@EdSchouten EdSchouten merged commit 34a33b4 into buildbarn:master Mar 8, 2024
1 check passed
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