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

Pass HOME to subprocesses #306

Merged
merged 3 commits into from
Dec 9, 2020
Merged

Pass HOME to subprocesses #306

merged 3 commits into from
Dec 9, 2020

Conversation

christopher-dG
Copy link
Member

@christopher-dG christopher-dG commented Dec 7, 2020

Closes #245

Lots of stuff needs HOME to work properly.

More concretely, both Git CLI and LibGit2 get their config from $HOME/.gitconfig by default, and that breaks when it's unset.

@christopher-dG
Copy link
Member Author

bors try

@christopher-dG
Copy link
Member Author

bors try

@bors
Copy link
Contributor

bors bot commented Dec 7, 2020

try

Already running a review

@codecov
Copy link

codecov bot commented Dec 7, 2020

Codecov Report

Merging #306 (9d60ae8) into master (043f2bf) will increase coverage by 0.12%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #306      +/-   ##
==========================================
+ Coverage   83.13%   83.26%   +0.12%     
==========================================
  Files          24       24              
  Lines        1281     1273       -8     
==========================================
- Hits         1065     1060       -5     
+ Misses        216      213       -3     
Impacted Files Coverage Δ
src/AutoMerge/guidelines.jl 83.33% <100.00%> (+0.72%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 043f2bf...9d60ae8. Read the comment docs.

bors bot added a commit that referenced this pull request Dec 7, 2020
@bors
Copy link
Contributor

bors bot commented Dec 8, 2020

try

Timed out.

@christopher-dG
Copy link
Member Author

I've been seeing SimpleMock hanging on nightly for another package, perhaps same thing here.

@DilumAluthge
Copy link
Member

I've been seeing SimpleMock hanging on nightly for another package, perhaps same thing here.

If you want, you can add a VERSION guard to only run the SimpleMock tests on Julia 1.5.

@christopher-dG
Copy link
Member Author

christopher-dG commented Dec 8, 2020

Yeah I'll probably do something like that. I'll get these PRs figured out tomorrow.

@christopher-dG
Copy link
Member Author

bors merge PLEASE I AM BEGGING YOU

bors bot added a commit that referenced this pull request Dec 8, 2020
306: Pass HOME to subprocesses r=christopher-dG a=christopher-dG

Closes #245 

Lots of stuff needs HOME to work properly.

More concretely, both Git CLI and LibGit2 get their config from `$HOME/.gitconfig` by default, and that breaks when it's unset.

Co-authored-by: Chris de Graaf <me@cdg.dev>
@bors
Copy link
Contributor

bors bot commented Dec 8, 2020

Build failed:

@christopher-dG
Copy link
Member Author

christopher-dG commented Dec 8, 2020

@DilumAluthge is there no way for me to not bump the version? (the current version is not released)

@DilumAluthge
Copy link
Member

Yeah, comment out the "VersionVigilante" line in the bors.toml file.

@christopher-dG
Copy link
Member Author

bors merge

bors bot added a commit that referenced this pull request Dec 8, 2020
306: Pass HOME to subprocesses r=christopher-dG a=christopher-dG

Closes #245 

Lots of stuff needs HOME to work properly.

More concretely, both Git CLI and LibGit2 get their config from `$HOME/.gitconfig` by default, and that breaks when it's unset.

Co-authored-by: Chris de Graaf <me@cdg.dev>
@christopher-dG
Copy link
Member Author

I am confused as to why this isn't merged now.

@DilumAluthge
Copy link
Member

Can you go to settings and uncheck VersionVigilante as a required status check?

@christopher-dG
Copy link
Member Author

I don't see any setting for that, unfortunately.

@DilumAluthge
Copy link
Member

(Settings -> branch protection -> master)

Anyway should be good now.

bors merge

@christopher-dG
Copy link
Member Author

Oh, I thought you mean in bors settings 🤦

@bors
Copy link
Contributor

bors bot commented Dec 9, 2020

Build succeeded:

  • continuous-integration/travis-ci/push

@bors bors bot merged commit 7f15fef into master Dec 9, 2020
@bors bors bot deleted the cdg/envs branch December 9, 2020 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LibGit2 callback error in automerge for private registry
3 participants