Susy: _support.scss: chamar call () com uma string tornou-se obsoleto no Sass 3.5 e será interrompido no Sass 4.0

Criado em 14 fev. 2017  ·  11Comentários  ·  Fonte: oddbird/susy

Eu descobri isso hoje quando atualizei para o Sass 3.5 e recebi vários avisos de depreciação. Haverá uma correção de código necessária da seguinte forma:

Linha 70 em susy / sass / susy / output / support / _support.scss

$_pass: call(unquote("#{$_type}-exists"), $_i);

precisa ser modificado para

$_pass: call(get-function(unquote("#{$_type}-exists")), $_i);

_Mais detalhes: http://blog.sass-lang.com/posts/809572-sass-35-release-candidate_

bug v2

Comentários muito úteis

Corrigido por 8e5dab9259077d6c0

Todos 11 comentários

Sim, para ser compatível com o pré e pós-3.5, precisamos fazer um pouco mais de solução alternativa. Corrigi isso em Susy 3 , espero que em breve, mas adoraria um PR para algo semelhante em Susy 2. Interessado em enviar um patch?

Certo. Na verdade, um novato no Github :-) Você tem algum procedimento específico a seguir para um patch ou existe um documento genérico online? Saúde!

Oh, tempo de aprendizado divertido para você! Tenho certeza de que há artigos, mas não sei de antemão. Também estou feliz em ajudar como posso. Em geral, você tem experiência com o Git? As etapas são (aproximadamente):

  1. "Fork" este repositório com o botão no topo.
  2. Em sua própria bifurcação do projeto, use o botão "clonar ou baixar" para obter o endereço e execute git clone <address> <local-directory> (substituindo aqueles) em uma janela de terminal para obter a versão local.
  3. Execute git branch -b <branch-name> para criar um novo branch
  4. faça suas alterações no diretório sass !! (adicione testes se puder, no test/sass dir)
  5. rake test para ter certeza de que todos os testes estão passando.
  6. git add .; git commit -m '<explain the commit>' para confirmar suas alterações
  7. git push -u origin <branch-name> para colocar tudo de volta no github
  8. Clique no botão new pull request em seu fork do github - selecione nosso repo / master-branch como a "base" e seu repo / new-branch como o "head"
  9. Envie sua solicitação de pull!

Então, eu dei a você muitos detalhes sobre as etapas do git sem muitos detalhes reais sobre as etapas do git ... Deixe-me saber se você tiver alguma dúvida ou se tiver algum problema. Estou feliz em expandir ou esclarecer se puder.

@mirisuzanne Isso precisa ser feito no branch susytwo para corrigi-lo para a v2?

@Jameskmonger está certo, embora eu não tenha certeza de que a solução acima seja a melhor (consulte http://oddbird.net/2017/03/30/safe-get/). Se você quiser fazer isso, ficarei feliz em considerar uma solicitação de pull! Obrigado!

Corrigido por 8e5dab9259077d6c0

Ei, sinto muito por reabrir este pedido @mirisuzanne e @nigelheap. Este problema ainda está de pé. Experimentou a mensagem hoje pela primeira vez. Eu apliquei o patch, mas ainda recebo o erro. Alguma ideia?

@brograhamer Você pode fornecer um exemplo simplificado (de preferência em um repositório GitHub) que demonstra o erro? Não consigo reproduzi-lo localmente.

Também não consigo reproduzir

@mirisuzanne trabalhando nisso, postarei de volta hoje com incompatibilidades com outros componentes, um patch ou um projeto simplificado para você verificar. Desculpe pelo atraso.

Editar: foi capaz de usar a v 2.2.14 de forma limpa sem um erro, o problema deve estar relacionado a alguma outra coisa em meu código. Ver o comentário do OP sobre isso me fez pensar que era relacionado a Susy, mas eu deveria ter feito meu próprio teste primeiro. Se eu encontrar um problema real com Susy, farei um novo tíquete. Desculpa!

Não há problema, é para isso que servem os exemplos simplificados - na metade das vezes, eles resolvem o problema. :) Obrigado pela atualização!

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

kyvour picture kyvour  ·  6Comentários

marzelin picture marzelin  ·  6Comentários

nmabhinandan picture nmabhinandan  ·  78Comentários

robsonsobral picture robsonsobral  ·  12Comentários

bs-thomas picture bs-thomas  ·  8Comentários