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

fix(visibility): Field access declared in a default class. #543

Merged
merged 1 commit into from
Feb 26, 2016
Merged

fix(visibility): Field access declared in a default class. #543

merged 1 commit into from
Feb 26, 2016

Conversation

GerardPaligot
Copy link
Contributor

Closes #245

@GerardPaligot
Copy link
Contributor Author

Wait for Jenkins builds before the merge of this PR.

@monperrus
Copy link
Collaborator

Closes #245? great! I wait for your ping.

@GerardPaligot
Copy link
Contributor Author

Jenkins is ok with Guava! :)

You can see it here (sorry, private link) and due to Guava, we're spooning 1 166 477 lines of code! :)

@monperrus
Copy link
Collaborator

Excellent, should I merge?

You can see it here
http://193.51.236.32:8080/view/Spoon%20Monitor%20View/ (sorry,
private link)
This view is really cool!, will it become the public one at
https://ci.inria.fr/sos/view/Spoon%20Monitor%20View/?

@GerardPaligot
Copy link
Contributor Author

Excellent, should I merge?

PR is okay for me.

This view is really cool!, will it become the public one at
https://ci.inria.fr/sos/view/Spoon%20Monitor%20View/?

There is work to do this.

@monperrus monperrus merged commit 3ff5986 into INRIA:master Feb 26, 2016
@monperrus
Copy link
Collaborator

There is work to do this.

Let's do it then.

@GerardPaligot GerardPaligot deleted the fix_245_2 branch February 26, 2016 10:21
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