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

Adds octokit samples #921

Merged
merged 8 commits into from Nov 4, 2015
Merged

Adds octokit samples #921

merged 8 commits into from Nov 4, 2015

Conversation

naveensrinivasan
Copy link

Implement LINQPad samples in NuGet packages. #505

http://www.linqpad.net/nugetsamples.aspx

  • Implement LINQPad samples
  • Update build to include the sample as nuget package
  • Write a test Change the build.fsx to do a lprun -compileonly check to make sure new check-in's don't break the samples.

@naveensrinivasan
Copy link
Author

The build failed due to #904

Naveen added 5 commits October 2, 2015 15:56
Fixed the args parameter to compile using lprun.exe
Including LINQPad.exe to compile the samples after every commit
This will include the samples in the nuget package.
Removed the integration test options because lprun has compileonly
option.
Validates Linqpad Samples as part of CI for every commit.
@naveensrinivasan
Copy link
Author

The build failed because of StopsMakingNewRequestsWhenTakeIsFulfilled

@naveensrinivasan naveensrinivasan mentioned this pull request Oct 5, 2015
@naveensrinivasan naveensrinivasan deleted the samples branch October 9, 2015 00:35
@naveensrinivasan naveensrinivasan restored the samples branch October 9, 2015 00:35
@naveensrinivasan naveensrinivasan changed the title [WIP] Adds octokit samples Adds octokit samples Oct 11, 2015
@shiftkey
Copy link
Member

shiftkey commented Nov 4, 2015

shiftkey added a commit that referenced this pull request Nov 4, 2015
@shiftkey shiftkey merged commit 302245b into octokit:master Nov 4, 2015
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.

None yet

2 participants