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

Fake references missing for IntegrationTests #1408

Merged
merged 6 commits into from
Jan 18, 2016

Conversation

dcorriveau-omniscient
Copy link
Contributor

Not sure if I'm doing something wrong but I didn't have FAKE references and so I couldn't compile the project.

@forki
Copy link
Member

forki commented Jan 18, 2016

What do you mean with "fake references"?
On Jan 18, 2016 20:30, "dcorriveau-omniscient" [email protected]
wrote:

Not sure if I'm doing something wrong but I didn't have FAKE references

and so I couldn't compile the project.

You can view, comment on, or merge this pull request online at:

#1408
Commit Summary

  • Find packages is now case-insensitive Github files (WIP #9) #93
  • Added FAKE reference to IntegrationTests project
  • Reverted previous fix
  • Added trailing space

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#1408.

@forki
Copy link
Member

forki commented Jan 18, 2016

Ah I understand. But please take the existing fake from the build group. No changes in dependencies or lock file needed. Only in the paket.references

@dcorriveau-omniscient
Copy link
Contributor Author

FakeLib doesn't work for the IntegrationTests project (exclamation mark next to the reference and I can't compile).
Edit: Alright, I'll try the paket.references only

@dcorriveau-omniscient
Copy link
Contributor Author

Alright, it should work with the Build group now.

@forki
Copy link
Member

forki commented Jan 18, 2016

Lgtm. Thx

forki added a commit that referenced this pull request Jan 18, 2016
Fake references missing for IntegrationTests
@forki forki merged commit 1273e0f into fsprojects:master Jan 18, 2016
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.

2 participants