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

Payroll: optimize by passing employee around as storage pointer #916

Merged
merged 1 commit into from Jul 11, 2019

Conversation

Projects
None yet
3 participants
@sohkai
Copy link
Member

commented Jul 10, 2019

TODO:

  • Change branch to master

@facuspagnuolo facuspagnuolo changed the base branch from payroll/add_min_acceptable_exchage_rates to master Jul 11, 2019

@facuspagnuolo facuspagnuolo force-pushed the payroll/pass-employee-storage branch from ebe64aa to ec9162d Jul 11, 2019

@facuspagnuolo facuspagnuolo merged commit 78ea592 into master Jul 11, 2019

6 checks passed

License Compliance All checks passed.
Details
Travis CI - Branch Build Passed
Details
Travis CI - Pull Request Build Passed
Details
WIP Ready for review
Details
coverage/coveralls First build on payroll/pass-employee-storage at 97.732%
Details
license/cla Contributor License Agreement is signed.
Details

@facuspagnuolo facuspagnuolo deleted the payroll/pass-employee-storage branch Jul 11, 2019

@coveralls

This comment has been minimized.

Copy link

commented Jul 11, 2019

Coverage Status

Coverage remained the same at 97.732% when pulling ec9162d on payroll/pass-employee-storage into 72736f9 on master.

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.