Skip to content
This repository has been archived by the owner on Sep 16, 2020. It is now read-only.

Develop #139

Open
wants to merge 30 commits into
base: master
Choose a base branch
from
Open

Develop #139

wants to merge 30 commits into from

Conversation

goleysunny
Copy link

No description provided.

pmonks and others added 30 commits March 12, 2018 13:59
Fix issue 104, this time in the CI environment as well as locally
Added Symphony Signals API support
Added javadoc comments, and signals example.
…t.ai.utils.AiSpellParser-Tests-from-Diffblue-Deeptest

Add tests for client.ai.utils.AiSpellParser from Diffblue Deeptest
…clients for key and pods. This included adding get/set for SymAuth session/key http clients.
complete.
2. Set 'active' flag on SymUser when calling getAllUsersWithDetails().
SymphonyClientConfig (in response to pull request comments).
@finos-admin
Copy link
Contributor

Thank you for your pull request and welcome to our community. We require contributors to be identified and we couldn't parse GitHub user details of the following contributors: Dov Katz, blackrock-engineering.

In order for your PR to be reviewed and merged, please follow the following directions:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails
  4. Amend the authors in the commit history; use git commit --amend to only change last commit, or one between git reset --soft and git rebase to checkout changes, rewrite commit history locally and (force) push changes to the downstream branch

Project team: please do not merge this Pull Request until Foundation staff have confirmed that a CLA is in place for the new contributor(s) listed above.

If you still have any questions or just don't know what to do next, please email [email protected]. /CC @finos-admin

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants