Skip to content
This repository was archived by the owner on Mar 10, 2020. It is now read-only.

Fix/lodash dependency #871

Closed
wants to merge 3 commits into from

Conversation

nGoline
Copy link
Contributor

@nGoline nGoline commented Oct 8, 2018

Must add lodash as a dependency because it's required in 2 files.

@alanshaw
Copy link
Contributor

alanshaw commented Oct 8, 2018

Hi @nGoline, thanks for the PR - our build tool generates changelogs, increments version numbers and adds contributors automatically - would you mind resubmitting with just the lodash dependency addition?

@nGoline
Copy link
Contributor Author

nGoline commented Oct 8, 2018

Hi @alanshaw. I just did it and the build is failing apparently with no regards to my code:
No such property: currentOS for class: groovy.lang.Binding

@nGoline
Copy link
Contributor Author

nGoline commented Oct 9, 2018

Ok, I'm having some problems with the size of my commit message.
I'll create a new branch and perform only one commit considering the actual change needed.

@alanshaw, I think you can delete this PR...

@nGoline nGoline closed this Oct 9, 2018
@nGoline nGoline deleted the fix/lodash-dependency branch October 9, 2018 13:33
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.

2 participants