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

[Merged by Bors] - Make lighthouse_version compatible with old Git #2223

Closed

Conversation

michaelsproul
Copy link
Member

Proposed Changes

When building the release binaries with Cross, Ubuntu 16.04 is used, which uses an old verison of Git lacking support for --exclude. This PR changes lighthouse_version to use --match instead.

@michaelsproul michaelsproul added ready-for-review The code is ready for review low-hanging-fruit Easy to resolve, get it before someone else does! labels Feb 24, 2021
Copy link
Member

@paulhauner paulhauner left a comment

Choose a reason for hiding this comment

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

ithinkthisshouldbemergedlol

@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Feb 24, 2021
@michaelsproul
Copy link
Member Author

bors r+

bors bot pushed a commit that referenced this pull request Feb 24, 2021
## Proposed Changes

When building the release binaries with Cross, Ubuntu 16.04 is used, which uses an old verison of Git lacking support for `--exclude`. This PR changes `lighthouse_version` to use `--match` instead.
@bors bors bot changed the title Make lighthouse_version compatible with old Git [Merged by Bors] - Make lighthouse_version compatible with old Git Feb 25, 2021
@bors bors bot closed this Feb 25, 2021
@michaelsproul michaelsproul deleted the fix-git-version-again branch February 25, 2021 00:54
michaelsproul added a commit that referenced this pull request Mar 10, 2021
## Proposed Changes

When building the release binaries with Cross, Ubuntu 16.04 is used, which uses an old verison of Git lacking support for `--exclude`. This PR changes `lighthouse_version` to use `--match` instead.
michaelsproul added a commit that referenced this pull request Mar 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low-hanging-fruit Easy to resolve, get it before someone else does! ready-for-merge This PR is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants