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

Update Enderecador.cs #40

Merged
merged 1 commit into from
Sep 12, 2015
Merged

Update Enderecador.cs #40

merged 1 commit into from
Sep 12, 2015

Conversation

SilverioMiranda
Copy link
Contributor

A propriedade precisa ter um get e um set definidos, não sei como você conseguiu compilar isso sem erros

A propriedade precisa ter um get e um set definidos, não sei como você conseguiu compilar isso sem erros
@adeniltonbs
Copy link
Member

Que versão do visual studio estais a utilizar?
Aqui no visual studio 2015 compila sem problemas, e a autoformatação do resharper 9 já elimina automaticamente esses setters por falta de uso.

Por favor me informe se está usando o visual studio 2013, pois quero manter retrocompatibilidade com ele.

Pelo que vi aqui http://blog.caelum.com.br/novidades-do-c-6-0/ e aqui https://github.com/dotnet/roslyn/wiki/New-Language-Features-in-C%23-6, isso são novidades do C# 6.0 que vem como default no VS2015.

Obs: Por curiosidade mudei aqui todos os projetos aqui para usar o C# 5.0 ao invés do default da IDE, mas também compilou normalmente com o C# 5.0 (bug no VS2015?)

De qualquer forma vou fazer um merge com suas alterações, pois quero manter compatibilidade com o VS2013, e pelo que vi aqui http://stackoverflow.com/questions/27093908/how-to-enable-c-sharp-6-0-feature-in-visual-studio-2013, o VS2013 só tem suporte a C# 6.0 preview que está bastante desatualizada em relação a versão final suportada apenas no VS 2015.

adeniltonbs added a commit that referenced this pull request Sep 12, 2015
@adeniltonbs adeniltonbs merged commit 7f60d5a into ZeusAutomacao:master Sep 12, 2015
@adeniltonbs
Copy link
Member

Alterei as definições do resharper para não recomendar a remoção dos setters nos casos em que isto pode ser feito por conta do C# 6.0 e mudei os setters para private (no commit 5bb9681), por enquanto já que não há classes que herdem dela.

E desde já obrigado pela colaboração.

@SilverioMiranda
Copy link
Contributor Author

Uso o 2013 e o .NET 4.0 para compatibilidade com windows XP.

Você já ativou o Code Contracts (http://research.microsoft.com/en-us/projects/contracts/) alguma vez para ver a quantidade de avisos que ele informa? Ele gasta aqui 14gb de RAM pra poder processar tudo e as informações sobre o código são top demais.
Se tiver afim de adicionar esse suporte ao projeto eu posso escrever os contratos aqui e ir enviando assim que for acabando.
Eu utilizo ele aqui para todos os outros projetos e ajuda demais a encontrar possíveis falhas que podem levar a comportamento inesperado

image

Obrigado

@adeniltonbs
Copy link
Member

Não o conhecia, vou dar uma olhada nele

marcosgerene pushed a commit that referenced this pull request Jul 5, 2018
marcosgerene added a commit that referenced this pull request Sep 4, 2019
guipiva-dev pushed a commit to BuildSolutions/DFe.NET that referenced this pull request Sep 30, 2024
…te-entrega

feat:Evento de Comprovante de Entrega para NFe (NT 2021.001)
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