Skip to content
This repository has been archived by the owner on Mar 19, 2019. It is now read-only.

Project name #437

Merged
merged 3 commits into from
Mar 17, 2017
Merged

Project name #437

merged 3 commits into from
Mar 17, 2017

Conversation

ddgenome
Copy link
Contributor

Ensure project object has an appropriate name.

Fixes #425 and #436 (in separate commits).

Clean up of TypeScript, Gherkin, and change log.

Make test quieter by removing/commenting out println's and console.log()'s.

@johnsonr
Copy link
Contributor

Would be good to merge #435 first, as there are probably conflicts.

David Dooling added 3 commits March 16, 2017 19:12
The project name used by the `generateWith()` testing method was
hardcoded to "project_name".  We now require a projectName parameter,
separate from the generator parameters.  The signature for
`generateWith()` is now

```typescript
generateWith(gen: ProjectGenerator, projectName: string, params?: {})
```

General cleanup of TypeScript and Gherkin formatting.

Closes #425
When creating the project artifact for testing, give it a name.

Clean up links in the change log.

Fixes #436
@ddgenome
Copy link
Contributor Author

#435 has been squash-merged and this has been rebased on it.

@johnsonr
Copy link
Contributor

LGTM

@johnsonr johnsonr merged commit 60cacfb into master Mar 17, 2017
@johnsonr johnsonr deleted the project-name branch March 17, 2017 01:02
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