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

Feature/18 default config and bootstrapping #19

Merged
merged 6 commits into from
Jun 3, 2015

Conversation

brettminnie
Copy link
Owner

Will close #18

@eddiejaoude
Copy link
Collaborator

Looks good so far :)

@brettminnie
Copy link
Owner Author

@eddiejaoude think this is ready for you

# BDBStudios\BreakfastSerializerTest\Fixtures\ExclusionClass:
# excludeVariables:
# - internalProperty
# - isExcluded
Copy link
Collaborator

Choose a reason for hiding this comment

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

Meant to be commented out?

Copy link
Owner Author

Choose a reason for hiding this comment

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

yes, hence the Example exclusions at the top :)

It will make more sense when #17 and #15 are implemented in the code base

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah ok :)

@brettminnie brettminnie force-pushed the feature/18_Default_Config_And_Bootstrapping branch from 98a1b1a to 0b919db Compare June 3, 2015 11:07
@brettminnie brettminnie force-pushed the feature/18_Default_Config_And_Bootstrapping branch from 0b919db to b184cfe Compare June 3, 2015 11:07
eddiejaoude added a commit that referenced this pull request Jun 3, 2015
…_Bootstrapping

Feature/18 default config and bootstrapping
@eddiejaoude eddiejaoude merged commit e54e5e6 into develop Jun 3, 2015
@eddiejaoude eddiejaoude deleted the feature/18_Default_Config_And_Bootstrapping branch June 3, 2015 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add default config and bootstrap
2 participants