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

Change percentagused to percentageused #903

Merged
merged 1 commit into from
Jul 8, 2020
Merged

Change percentagused to percentageused #903

merged 1 commit into from
Jul 8, 2020

Conversation

phil-davis
Copy link
Contributor

IMO these names can be fixed without created a BC-break problem. The "magic" numbers are still 10 and 11.

But happy to be told where the names are used elsewhere.

@phil-davis phil-davis requested a review from labkode as a code owner June 29, 2020 14:25
@butonic
Copy link
Contributor

butonic commented Jun 30, 2020

@labkode this might need testing on your side

@labkode
Copy link
Member

labkode commented Jul 1, 2020

@phil-davis this need to be changed at the source.
@ffurano can you bring this fix to the EOS repo?

@ffurano
Copy link
Contributor

ffurano commented Jul 1, 2020

yes, no probs

@ffurano
Copy link
Contributor

ffurano commented Jul 1, 2020

done

@labkode
Copy link
Member

labkode commented Jul 1, 2020

@ffurano can you update the copy of proto in reva with the new version and adapt the code accordingly?

ffurano pushed a commit to ffurano/reva that referenced this pull request Jul 1, 2020
@ffurano
Copy link
Contributor

ffurano commented Jul 1, 2020

Just made a PR for the compiled go proto

@phil-davis
Copy link
Contributor Author

Rebased just now to get fresh CI...

@labkode labkode merged commit 07ab188 into cs3org:master Jul 8, 2020
@phil-davis phil-davis deleted the percentageused branch July 8, 2020 09:52
ffurano pushed a commit to ffurano/reva that referenced this pull request Jul 9, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Aug 20, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Sep 9, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Oct 5, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Oct 6, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Oct 20, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Oct 23, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Nov 10, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Nov 13, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Dec 2, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Dec 2, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Dec 2, 2020
ishank011 pushed a commit to ishank011/reva that referenced this pull request Dec 2, 2020
ffurano pushed a commit to ffurano/reva that referenced this pull request Dec 3, 2020
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.

4 participants