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

Possible solution for #1077 #1715

Closed
wants to merge 22 commits into from
Closed

Possible solution for #1077 #1715

wants to merge 22 commits into from

Conversation

AndrewDryga
Copy link

No description provided.

chriseppstein and others added 20 commits December 4, 2013 15:10
Fix invalid method call for strip_trailing_seperator in configuration
This reverts commit af1892b, reversing
changes made to 587c2e5.
```
$ compass create myproject
directory myproject/
directory myproject/sass/
directory myproject/stylesheets/
   create myproject/config.rb
   create myproject/sass/screen.scss
   create myproject/sass/print.scss
   create myproject/sass/ie.scss
   create myproject/stylesheets/print.css
   create myproject/stylesheets/screen.css
   create myproject/stylesheets/ie.css
```
@AndrewDryga
Copy link
Author

Spent few hours on ruby-lang IRC (thanks to darix) channel to debug it and i found monkey patch:
https://gist.github.com/AndrewDryga/529b3883c59116f71940#file-fix2-rb

@scottdavis
Copy link
Member

Is this an issue on the alpha release?

@AndrewDryga
Copy link
Author

Not only on alpha, it exist for a long time. Look into #1077 comments and links

@scottdavis
Copy link
Member

can you make sure i think this commit fixed in on master c109bf8

@AndrewDryga
Copy link
Author

@scottdavis yes, you are right, it's fixed there. Can you merge it into --pre version?

Because right now it have this bug included..

@scottdavis
Copy link
Member

which pre version are you on?

@AndrewDryga
Copy link
Author

Compass 1.0.0.alpha.20

@AndrewDryga
Copy link
Author

@chriseppstein its really bad practice to close something without comments. Is it fixed in some version? Maybe I could test solution? Or you just closed ticket/merge request?

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

5 participants