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

Bump to latest zinc in order to consume zinc analysis fix #7854

Merged

Conversation

Projects
None yet
2 participants
@stuhood
Copy link
Member

commented Jun 5, 2019

Problem

We need to release #7853, which fixes #7850.

Solution

Bump to the version built in #7853, and modify a test to confirm that it is fixed.

@stuhood stuhood requested review from wisechengyi, illicitonion and blorente Jun 5, 2019

@stuhood stuhood added this to the 1.17.x milestone Jun 5, 2019

@stuhood

This comment was marked as outdated.

Copy link
Member Author

commented Jun 5, 2019

(have canceled CI, because this will not pass until #7853 is landed and published.)

@stuhood stuhood force-pushed the twitter:stuhood/analysis-path-hardcoding-fix-2 branch from 3cd62cc to 9eb1d30 Jun 5, 2019

@stuhood stuhood merged commit ab179a2 into pantsbuild:master Jun 6, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@stuhood stuhood deleted the twitter:stuhood/analysis-path-hardcoding-fix-2 branch Jun 6, 2019

stuhood added a commit that referenced this pull request Jun 7, 2019

Bump to latest zinc in order to consume zinc analysis fix (#7854)
### Problem

We need to release #7853, which fixes #7850.

### Solution

Bump to the version built in #7853, and modify a test to confirm that it is fixed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.