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

Move trailing slashes config #67

Merged
merged 1 commit into from
Feb 1, 2015
Merged

Conversation

dustinfarris
Copy link
Owner

The trailing slashes config isn't really something that changes between environments, so it shouldn't be configured as such.

This PR redefines how addTrailingSlashes is configured.

This is a breaking change.

Rather than making this an environment variable, make it a property of
the adapter itself that can be easily configured by extending the
adapter.
benkonrath added a commit that referenced this pull request Feb 1, 2015
@benkonrath benkonrath merged commit 59db0b5 into master Feb 1, 2015
@benkonrath
Copy link
Collaborator

This means the next release will be 0.6.0, right?

@dustinfarris dustinfarris deleted the move-trailing-slashes-config branch February 1, 2015 14:12
@dustinfarris
Copy link
Owner Author

@benkonrath I'm not sure. Since the rewrite, it has been my intention to mimic ember-cli's versioning as much as possible—which is not exactly semver compliant. I think once we release 1.0.0, versioning will become much more important—and that won't happen until ED 1.0 is released and we've had a chance to battle test it. For now, I suppose 0.5 represents EDA post-rewrite, and I'd like to keep it at that minor version until 1.0, or when/if we feel like we've reached some other important milestone.

What do you think?

@benkonrath
Copy link
Collaborator

I guess semver gives us flexibility before 1.0.0 so I'm good with your proposal. In general I trust your intuition and judgment about when and how to do the releases. I was really just curious about how we would be proceeding.

@dustinfarris
Copy link
Owner Author

Ok, so let's call 0.6 a pre-ED-1.0 stable release of EDA—whenever we feel like we've addressed all of the major outstanding issues/bugs.

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