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

Project skeleton #3

Merged
merged 2 commits into from Mar 21, 2019
Merged

Conversation

jskeet
Copy link
Member

@jskeet jskeet commented Mar 21, 2019

There's no "interesting" code yet.

Creating these now makes it easier to add code in future easy-to-review PRs.
… tests

Other alternatives considered:

- Publish a NuGet package
- Copy the whole project rather than including it within the IntegrationTests project
- Use git submodules

Happy to discuss pros/cons of each of these
Copy link

@chrisdunelm chrisdunelm left a comment

Choose a reason for hiding this comment

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

LGTM
The code copying isn't ideal, but it can be changed later if it becomes a problem.

@jskeet jskeet merged commit 9c511d1 into GoogleCloudPlatform:master Mar 21, 2019
@jskeet jskeet deleted the project-skeleton branch March 21, 2019 14:53
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.

None yet

2 participants