Skip to content

Conversation

@melan
Copy link

@melan melan commented May 24, 2019

fixes #32, also makes package.json source of truth for things like name and version

@salesforce-cla
Copy link

Thanks for the contribution! It looks like @melan is an internal user so signing the CLA is not required. However, we need to confirm this.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling c660b0c on melan:master into 7dfb8e8 on salesforce:master.

@iamigo iamigo requested review from iamigo, jgraff2 and pallavi2209 May 24, 2019 16:28
@melan
Copy link
Author

melan commented May 24, 2019

I filled the CLA form

Copy link
Contributor

@pallavi2209 pallavi2209 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, for some reason SLA still says "missing". I will wait for @jgraff2 to approve since he is most familiar with this repo.

@iamigo
Copy link
Contributor

iamigo commented Sep 17, 2019

@jgraff2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

improve building contextDefinition

4 participants